[Mlir-commits] [mlir] [MLIR][NVVM] Add support for st.bulk Op (PR #131727)

[Mehdi Amini]

================
@@ -160,6 +160,12 @@ LogicalResult CvtFloatToTF32Op::verify() {
   return success();
 }
 
+LogicalResult BulkStoreOp::verify() {
+  if (getInitVal() != 0)
+    return emitOpError("only 0 is supported for initVal in st.bulk");
----------------
joker-eph wrote:

```suggestion
    return emitOpError("only 0 is supported for initVal, got ") << getInitVal();
```

You don't need to report the op name, it'll be prepended to the error message, and also adding the current value alongside the expected one is helpful for debugging in general.

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