[PATCH] D12246: [NVPTX] change threading intrinsics from noduplicate to convergent

Xuetian Weng via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 30 14:22:35 PDT 2015


wengxt marked an inline comment as done.

================
Comment at: lib/Transforms/Scalar/JumpThreading.cpp:276
@@ -275,3 +275,3 @@
     if (const CallInst *CI = dyn_cast<CallInst>(I)) {
-      if (CI->cannotDuplicate())
+      if (CI->cannotDuplicate() || CI->hasFnAttr(Attribute::Convergent))
         // Blocks with NoDuplicate are modelled as having infinite cost, so they
----------------
arsenm wrote:
> Can you factor this into a CI->isConvergent()?
This part is separated and done in D12484


http://reviews.llvm.org/D12246





More information about the llvm-commits mailing list