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

[Guray Ozen]

================
@@ -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");
----------------
grypp wrote:

I am also not clear why do we relax this check. 

@durga4github we can have mandatory id and have `DefaultValuedAttr`? 

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