jdoerfert created this revision.
jdoerfert added a reviewer: tra.
Herald added subscribers: bollu, yaxunl.
Herald added a project: clang.
jdoerfert added a child revision: D77240: [CUDA] Add missing cmath overloads.

The other macro uses a unqualified lookup already and the qualified one
will cause problems in the OpenMP overlay.

Depends on D77238 <https://reviews.llvm.org/D77238>.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D77239

Files:
  clang/lib/Headers/__clang_cuda_cmath.h


Index: clang/lib/Headers/__clang_cuda_cmath.h
===================================================================
--- clang/lib/Headers/__clang_cuda_cmath.h
+++ clang/lib/Headers/__clang_cuda_cmath.h
@@ -191,7 +191,7 @@
       typename __clang_cuda_enable_if<std::numeric_limits<__T>::is_integer,    
\
                                       __retty>::type                           
\
       __fn(__T __x) {                                                          
\
-    return ::__fn((double)__x);                                                
\
+    return __fn((double)__x);                                                  
\
   }
 
 // Defines an overload of __fn that accepts one two arithmetic arguments, calls


Index: clang/lib/Headers/__clang_cuda_cmath.h
===================================================================
--- clang/lib/Headers/__clang_cuda_cmath.h
+++ clang/lib/Headers/__clang_cuda_cmath.h
@@ -191,7 +191,7 @@
       typename __clang_cuda_enable_if<std::numeric_limits<__T>::is_integer,    \
                                       __retty>::type                           \
       __fn(__T __x) {                                                          \
-    return ::__fn((double)__x);                                                \
+    return __fn((double)__x);                                                  \
   }
 
 // Defines an overload of __fn that accepts one two arithmetic arguments, calls
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D77239: [CUDA][N... Johannes Doerfert via Phabricator via cfe-commits

Reply via email to