search

hughperkins / clnn

OpenCL backend for Torch nn neural networks library
BSD 2-Clause "Simplified" License
126 stars 16 forks source link

comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') #14

Closed linkerlin closed 8 years ago

linkerlin commented 8 years ago

luajit -l clnn -e 'clnn.test()' Running 56 tests |___ ==> Abs_backwardUsing Apple , OpenCL platform: Apple Using OpenCL device: Iris Pro |___ ==> ClassNLLCriterionMultipleTargetTHClReduceAll.cl build log:

:9:10: warning: unused variable 'in1' float *in1 = &_in1; ^ :10:10: warning: unused variable 'out' float *out = &_out; ^ _______________|________________________________________ ==> SoftMax_backward/tmp/luarocks_clnn-scm-1-2347/clnn/SoftMax.cpp build log: :38:20: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') for (int i=0; i:63:20: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') for (int i=0; i:38:20: warning: comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') for (int i=0; i mseApply_3t_0s_0pt_-2_-2_-2__out = 0.00040675208460443 \* (_in1 - _in2) build log: :37:12: warning: double precision constant requires cl_khr_fp64, casting to single precision *out = 0.00040675208460443 \* (_in1 - *in2); ^ Apply_3t_0s_0pt_-2_-2_-2__out = 0.00043487714720591 \* (_in1 - _in2) build log: :37:12: warning: double precision constant requires cl_khr_fp64, casting to single precision *out = 0.00043487714720591 \* (_in1 - *in2); ^ ________________________________________________________ ==> Done Completed 88 asserts in 56 tests with 0 errors ---
hughperkins commented 8 years ago

Ok. warnings about unsigned/signed comparison removed from SoftMax.cl in 27bd0df

hughperkins commented 8 years ago

I guess this can be closed, right?