[PATCH] D12246: [NVPTX] change threading intrinsics from noduplicate to convergent
Jingyue Wu via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 24 12:15:18 PDT 2015
jingyue accepted this revision.
This revision is now accepted and ready to land.
================
Comment at: test/CodeGen/NVPTX/convergent-syncthreads.ll:3
@@ +2,3 @@
+
+; Make sure the call to syncthreads is not duplicate here by the LLVM
+; optimizations, because it has the convergent attribute set.
----------------
I understand that you are checking against JumpThreading duplicating syncthreads, but the wording is inaccurate.
The convergent attribute restricts the compiler so that it moves a convergent instruction only to a control-equivalent location. It does _not_ prevent LLVM from duplicating convergent instructions.
http://reviews.llvm.org/D12246
More information about the llvm-commits
mailing list