[Mlir-commits] [llvm] [mlir] [LLVM][NVPTX][MLIR] Add mbarrier layout support (PR #217252)

[Durgadoss R]

================
@@ -2157,9 +2157,23 @@ foreach is_shared = [true, false] in {
   }
 }
 
+def int_nvvm_mbarrier_init_layout :
+  DefaultAttrsIntrinsic<[], [llvm_shared_ptr_ty,  // mbar
+                              llvm_i32_ty,        // count
+                              llvm_i32_ty],       // layout
----------------
durga4github wrote:

We seem to have two ways to init the legacy (layout-v0) mbar now. That may get confusing..So,
a)
either we can add this layout as an arg in the existing init and Auto-upgrade
b)
Just add a new intrinsic with `init_layout_v1` in the name itself. 

b) seems simpler to me.

Another note:
Let us use any_ptr_ty for the input to support both generic as well as shared-cta addresses.
Once the constraint check infra is in place, we can narrow it down to just the two.




https://github.com/llvm/llvm-project/pull/217252