[Mlir-commits] [mlir] [mlir][LLVMIR] Add operand bundle support for llvm.intr.assume (PR #112143)

Tobias Gysi llvmlistbot at llvm.org
Mon Oct 14 01:26:22 PDT 2024


================
@@ -68,6 +68,12 @@ static LogicalResult convertIntrinsicImpl(OpBuilder &odsBuilder,
   if (isConvertibleIntrinsic(intrinsicID)) {
     SmallVector<llvm::Value *> args(inst->args());
     ArrayRef<llvm::Value *> llvmOperands(args);
+
+    SmallVector<llvm::OperandBundleUse> opBundles;
----------------
gysit wrote:

```suggestion
    SmallVector<llvm::OperandBundleUse> llvmOpBundles;
```
nit: should we also use llvmOpBundles similar to llvmOperands?

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


More information about the Mlir-commits mailing list