[PATCH] D17657: [NVPTX] Add intrinsics to support named barriers.

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 18:23:15 PDT 2016


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM.


================
Comment at: include/llvm/IR/IntrinsicsNVVM.td:743
@@ -740,1 +742,3 @@
+  def int_nvvm_barrier : GCCBuiltin<"__nvvm_bar">,
+      Intrinsic<[], [llvm_i32_ty, llvm_i32_ty], [IntrNoDuplicate, IntrConvergent]>;
   def int_nvvm_barrier0_popc : GCCBuiltin<"__nvvm_bar0_popc">,
----------------
arpith-jacob wrote:
> Done.  Thanks for the explanation; makes sense.  When you do remove noduplicate, please go ahead and remove it for named barriers as well.  Thanks!
This happened in r264107. Update this patch before submitting.


http://reviews.llvm.org/D17657





More information about the llvm-commits mailing list