Skip to content

Commit

Permalink
Fix MSVC build.
Browse files Browse the repository at this point in the history
In commit 33be1b1, I updated rounding_mode.* to their current
OpenCL-CTS versions without noticing that the old versions had an
intentional deviation from OpenCL-CTS: a call to vlog_error had been
commented out, and the headers that were needed only for that were also
left out. This commit reapplies those changes.
  • Loading branch information
hvdijk committed Nov 14, 2024
1 parent ead7474 commit 824a310
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 7 deletions.
2 changes: 0 additions & 2 deletions oclmath/rounding_mode.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -130,8 +130,6 @@ RoundingMode set_round(RoundingMode r, Type outType) {

int err = _controlfp_s(&oldRound, 0, 0); // get rounding mode into oldRound
if (err) {
vlog_error("\t\tERROR: -- cannot get rounding mode in %s:%d\n", __FILE__,
__LINE__);
return kDefaultRoundingMode; // what else never happens
}

Expand Down
5 changes: 0 additions & 5 deletions oclmath/rounding_mode.h
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,6 @@

#include "compat.h"

#if (defined(_WIN32) && defined(_MSC_VER))
#include "errorHelpers.h"
#include "testHarness.h"
#endif

typedef enum {
kDefaultRoundingMode = 0,
kRoundToNearestEven,
Expand Down

0 comments on commit 824a310

Please sign in to comment.