[Mlir-commits] [mlir] [mlir][LLVM] Add operand bundle support (PR #108933)

Markus Böck llvmlistbot at llvm.org
Mon Sep 23 09:31:41 PDT 2024


================
@@ -1333,11 +1487,35 @@ ParseResult CallOp::parse(OpAsmParser &parser, OperationState &result) {
       return failure();
   }
 
+  auto opBundlesLoc = parser.getCurrentLocation();
+  if (auto result = parseOpBundles(parser, opBundleOperands,
+                                   opBundleOperandTypes, opBundleTags);
+      result.has_value() && failed(*result))
----------------
zero9178 wrote:

```suggestion
      result && failed(*result))
```
I think in LLVM/MLIR its more conventional to rely on implicit/contextual bool conversion

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


More information about the Mlir-commits mailing list