-
Notifications
You must be signed in to change notification settings - Fork 343
Open
Description
If you want to support Visual Studio's OpenMP which is compiler flag"/openmp" , it has been at version OpenMP 2.0 for around a decade, and complains about size_t for the index and the error says it needs to be a signed int.
tinyraytracer/tinyraytracer.cpp
Lines 123 to 124 in 5888c49
for (size_t j = 0; j<height; j++) { // actual rendering loop | |
for (size_t i = 0; i<width; i++) { |
So the above size_t index works when it changed to a signed int for VS compilers.
I didn't want to make a pull since I'm not quite sure what the best way to add "/openmp" to how you have your CMakeLists.txt arranged for Visual Studio.
Metadata
Metadata
Assignees
Labels
No labels