search

clMathLibraries / clBLAS

a software library containing BLAS functions written in OpenCL
Apache License 2.0
843 stars 237 forks source link

Add small helper function to build up kernel build options #38

Closed gicmo closed 10 years ago

gicmo commented 10 years ago

Using this helper instead of the old strcat will avoid having two spaces between kernel build options and thus fix the CL_INVALID_BUILD_OPTIONS errors of issue #37)

gicmo commented 10 years ago

Ah, strlcat is not available on Linux (Thanks Ulrich ;) [1]), I will replace it with a call to strncat.

[1] http://en.wikibooks.org/wiki/C_Programming/C_Reference/nonstandard/strlcpy

kknox commented 10 years ago

Such an interesting bug on OSX; thank you for fixing the build error. I will try to get this merged in today or tomorrow, after a short review.