[PATCH] D64657: Fix missing use of defined() in include guard in AMDGPU target

[David Tenty via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL365952: Fix missing use of defined() in include guard (authored by daltenty, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D64657?vs=209557&id=209576#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64657/new/

https://reviews.llvm.org/D64657

Files:
  llvm/trunk/lib/Target/AMDGPU/AMDGPULibCalls.cpp


Index: llvm/trunk/lib/Target/AMDGPU/AMDGPULibCalls.cpp
===================================================================
--- llvm/trunk/lib/Target/AMDGPU/AMDGPULibCalls.cpp
+++ llvm/trunk/lib/Target/AMDGPU/AMDGPULibCalls.cpp
@@ -865,7 +865,7 @@
 
 namespace llvm {
 static double log2(double V) {
-#if _XOPEN_SOURCE >= 600 || _ISOC99_SOURCE || _POSIX_C_SOURCE >= 200112L
+#if _XOPEN_SOURCE >= 600 || defined(_ISOC99_SOURCE) || _POSIX_C_SOURCE >= 200112L
   return ::log2(V);
 #else
   return log(V) / 0.693147180559945309417;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64657.209576.patch
Type: text/x-patch
Size: 528 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190712/a4cc3f4c/attachment.bin>