[PATCH] D42411: [test-suite] Fix ambigous call to overloaded function isnan
Simon Dardis via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 25 08:10:58 PST 2018
sdardis updated this revision to Diff 131458.
sdardis added a comment.
Update as per @MatzeB 's comment, checking before commit showed it broke the build on OSX.
Repository:
rT test-suite
https://reviews.llvm.org/D42411
Files:
MultiSource/Benchmarks/DOE-ProxyApps-C++/CLAMR/clamr_cpuonly.cpp
Index: MultiSource/Benchmarks/DOE-ProxyApps-C++/CLAMR/clamr_cpuonly.cpp
===================================================================
--- MultiSource/Benchmarks/DOE-ProxyApps-C++/CLAMR/clamr_cpuonly.cpp
+++ MultiSource/Benchmarks/DOE-ProxyApps-C++/CLAMR/clamr_cpuonly.cpp
@@ -55,7 +55,7 @@
*/
#include <algorithm>
-#include <math.h>
+#include <cmath>
#include <stdio.h>
#include <stdlib.h>
#include <sys/time.h>
@@ -499,7 +499,7 @@
int error_status = STATUS_OK;
- if (isnan(H_sum)) {
+ if (std::isnan(H_sum)) {
printf("Got a NAN on cycle %d\n",ncycle);
error_status = STATUS_NAN;
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42411.131458.patch
Type: text/x-patch
Size: 626 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180125/c6fdd4ff/attachment.bin>
More information about the llvm-commits
mailing list