byzhang / cudpp

Automatically exported from code.google.com/p/cudpp
Other
0 stars 0 forks source link

make issues: missing typinfo, cstdlib, etc. includes in cutil and testrig #31

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi,

1) cudpp ships with precompiled libcutil.a, but compiling example apps 
fails due to it being in a wrong format. I guess it is compiled for 32bit 
host, and i am running 64 bit. So cudpp make, or some kind other make 
should rebuild it. Running make in common/ fails with

./../common/inc/cmd_arg_reader.h: In member function ‘const T* 
CmdArgReader::getArgHelper(const std::string&)’:
./../common/inc/cmd_arg_reader.h:416: error: must #include <typeinfo> 
before using typeid
./../common/inc/cmd_arg_reader.h:432: error: must #include <typeinfo> 
before using typeid
src/cmd_arg_reader.cpp: In destructor ‘CmdArgReader::~CmdArgReader()’:
src/cmd_arg_reader.cpp:101: error: must #include <typeinfo> before using 
typeid
src/cmd_arg_reader.cpp:106: error: must #include <typeinfo> before using 
typeid
src/cmd_arg_reader.cpp:111: error: must #include <typeinfo> before using 
typeid
src/cmd_arg_reader.cpp:116: error: must #include <typeinfo> before using 
typeid
src/cmd_arg_reader.cpp:121: error: must #include <typeinfo> before using 
typeid
make: *** [obj/release/cmd_arg_reader.cpp_o] Error 1

2) If i change anything in kernels i hava to manualy delete compiled .o 
files, make doesn;t recreate them automaticaly.

3) Building testrig fails with

In file included from spmvmult_gold.cpp:13:
sparse.h: In constructor ‘MMMatrix::MMMatrix(unsigned int, unsigned int, 
unsigned int)’:
sparse.h:46: error: ‘malloc’ was not declared in this scope
spmvmult_gold.cpp: In function ‘void readMatrixMarket(MMMatrix*, const 
char*)’:
spmvmult_gold.cpp:94: error: ‘exit’ was not declared in this scope
spmvmult_gold.cpp:122: error: ‘qsort’ was not declared in this scope
make: *** [obj/release/spmvmult_gold.cpp_o] Error 1

Original issue reported on code.google.com by korgulec@gmail.com on 22 Jul 2009 at 1:38

GoogleCodeExporter commented 9 years ago
Hi, 
this patch should fix 1) and 3)

Original comment by korgulec@gmail.com on 22 Jul 2009 at 7:41

Attachments:

GoogleCodeExporter commented 9 years ago
I don't get the errors in #1 or #3 on 64-bit Ubuntu 8.04, but since your patch 
is 
harmless, I will go ahead and apply it.  If #2 is really an issue (I don't 
think it is 
a big deal, I'm pretty sure make handles the dependencies OK unless you edit 
one of the 
_kernel or _cta files without touching the _app file), we can open another 
issue for 
it.

Thanks for the patch and the issue report!

Original comment by harr...@gmail.com on 24 Jul 2009 at 6:46

GoogleCodeExporter commented 9 years ago
This issue was closed by r7.

Original comment by harr...@gmail.com on 24 Jul 2009 at 6:49