search

tromp / cuckoo

a memory-bound graph-theoretic proof-of-work system
Other
822 stars 173 forks source link

make cuda28 fails #20

Closed ctcyang closed 7 years ago

ctcyang commented 7 years ago

The latest commit I could get working was af7c892. The error message I got was:

$ make cuda28
nvcc -o cuda28 -DEDGEBITS=27 -arch sm_35 cuda_miner.cu -lssl -lcrypto
cuda_miner.cu(136): error: identifier "HALFSIZE" is undefined
cuda_miner.cu(162): error: identifier "SIZE" is undefined
cuda_miner.cu(169): error: identifier "SIZESHIFT" is undefined
cuda_miner.cu(170): error: identifier "SIZE" is undefined
cuda_miner.cu(187): error: identifier "SIZE" is undefined
cuda_miner.cu(195): error: identifier "SIZESHIFT" is undefined
cuda_miner.cu(215): error: argument of type "siphash_keys *" is incompatible with parameter of type "const char *"
cuda_miner.cu(215): error: argument of type "char *" is incompatible with parameter of type "u32"
cuda_miner.cu(215): error: too few arguments in function call
cuda_miner.cu(224): error: identifier "HALFSIZE" is undefined
cuda_miner.cu(242): error: identifier "HALFSIZE" is undefined
cuda_miner.cu(260): error: identifier "SIZESHIFT" is undefined
cuda_miner.cu(309): error: identifier "SIZESHIFT" is undefined
cuda_miner.cu(321): error: identifier "HALFSIZE" is undefined
cuda_miner.cu(365): error: identifier "SIZE" is undefined
15 errors detected in the compilation of "/tmp/tmpxft_00001830_00000000-9_cuda_miner.cpp1.ii".
Makefile:132: recipe for target 'cuda28' failed
make: *** [cuda28] Error 2
tromp commented 7 years ago

dear ctcyang,

sorry about that. I changed cuckoo.h and forgot to test cuda.cu Thanks for pointing out that I broke it. Should be fixed now...

regards, -John

ctcyang commented 7 years ago

Yep, it compiles for me now. Thanks for the quick fix!