hughperkins / DeepCL

OpenCL library to train deep convolutional neural networks
Mozilla Public License 2.0
866 stars 200 forks source link

Add cstdlib for compilation on OS X #2

Closed kennethban closed 9 years ago

kennethban commented 9 years ago

Hi,

Thanks for this wonderful resource.

I had problems compiling DeepCL on OS X Yosemite as it kept failing:

[ 5%] Building CXX object CMakeFiles/DeepCL.dir/src/Translator.cpp.o /Users/kennethban/Development/DeepCL/src/Translator.cpp:23:43: error: call to 'abs' is ambiguous const int rowCopyLength = imageSize - abs( translateCols ); ^~~ /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/cmath:660:1: note: candidate function abs(float x) _NOEXCEPT {return fabsf(x);} ^ /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/cmath:664:1: note: candidate function abs(double x) _NOEXCEPT {return fabs(x);} ^ /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/cmath:668:1: note: candidate function abs(long double __x) _NOEXCEPT {return fabsl(__x);} ^ 1 error generated.

This was fixed by including the cstdlib for OS X

Thanks, Kenneth

hughperkins commented 9 years ago

Merged. Thanks! :-)

kennethban commented 9 years ago

Thanks for this great project. Finally I can use the AMD GPUs on the Mac.

Looking forward to learning more. I’m a newbie at this

On 10 Apr 2015, at 6:59 pm, Hugh Perkins notifications@github.com wrote:

Merged. Thanks! :-)

— Reply to this email directly or view it on GitHub https://github.com/hughperkins/DeepCL/pull/2#issuecomment-91516748.