[Mlir-commits] [mlir] [MLIR][NVVM] Adds an explicit aligned boolean attribute to nvvm.barrier (PR #192203)

[Durgadoss R]

================
@@ -2916,9 +2917,8 @@ LogicalResult NVVM::SetMaxRegisterOp::verify() {
 }
 
 LogicalResult NVVM::BarrierOp::verify() {
-  if (getNumberOfThreads() && !getBarrierId())
-    return emitOpError(
-        "barrier id is missing, it should be set between 0 to 15");
----------------
durga4github wrote:

I think it's better to enforce it, as it aligns with PTX syntax.
What do we gain by relaxing this?

Aside, (not for this PR)
Looking at the latest set of intrinsics available, I feel that having bar-ID as a mandatory field would simplify these Verifier checks. @grypp , thoughts?

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