eyalroz / cuda-api-wrappers

Thin, unified, C++-flavored wrappers for the CUDA APIs
BSD 3-Clause "New" or "Revised" License
785 stars 80 forks source link

Compiler Flags Incompatibilities #71

Closed DeveloperPaul123 closed 5 years ago

DeveloperPaul123 commented 5 years ago

Quite a few flags are set by default in the main CMakeLists.txt. I guess I can preface this issue by asking if multiple compilers are meant to be supported?

If so, then equivalent flags for other compilers (i.e. MSVC) need to be used instead of the ones currently being set. For example, MSVC does not support the -Wpendantic flag and setting this actually results in a compilation error.

Would something like this work?

eyalroz commented 5 years ago

@DeveloperPaul123 : Please try the development branch, where I've committed what is hopefully a fix.

DeveloperPaul123 commented 5 years ago

@eyalroz You changes worked for me.