This revision was automatically updated to reflect the committed changes.
Closed by commit rL272299: [CUDA] Implement __shfl* intrinsics in clang
headers. (authored by jlebar).
Changed prior to commit:
http://reviews.llvm.org/D21162?vs=60223=60230#toc
Repository:
rL LLVM
jlebar added inline comments.
Comment at: lib/Headers/__clang_cuda_intrinsics.h:77-80
@@ +76,6 @@
+_Static_assert(sizeof(__tmp) == sizeof(__in));
\
+memcpy(&__tmp, &__in, sizeof(__in));
\
+__tmp =
jlebar updated this revision to Diff 60223.
jlebar marked 2 inline comments as done.
jlebar added a comment.
Update after tra's review.
http://reviews.llvm.org/D21162
Files:
include/clang/Basic/BuiltinsNVPTX.def
lib/Headers/__clang_cuda_intrinsics.h
tra added inline comments.
Comment at: lib/Headers/__clang_cuda_intrinsics.h:77-80
@@ +76,6 @@
+_Static_assert(sizeof(__tmp) == sizeof(__in));
\
+memcpy(&__tmp, &__in, sizeof(__in));
\
+__tmp =
jlebar added a comment.
Thank you for the reviews, Justin!
http://reviews.llvm.org/D21162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jlebar added a comment.
(Art, I would appreciate a second set of eyes on this one, as the last time I
did this -- with ldg -- I messed up pretty badly.)
http://reviews.llvm.org/D21162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
jholewinski added a comment.
Looks reasonable to me.
http://reviews.llvm.org/D21162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jlebar created this revision.
jlebar added a reviewer: tra.
jlebar added subscribers: cfe-commits, jholewinski.
Clang changes to make use of the LLVM intrinsics added in D21160.
http://reviews.llvm.org/D21162
Files:
include/clang/Basic/BuiltinsNVPTX.def
lib/Headers/__clang_cuda_intrinsics.h