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

[Sirui Mu]

Lancern wrote:

> What are those burdens?

The main problem with properties is that they are not available through `Operation *`. You have to cast it to a `mlir::LLVM::FooOp` before getting/setting properties. However, when importing and exporting intrinsics, we all operate on `Operation *` and thus storing operand bundle tags in properties could introduce a bit trouble in such a case.

> Might also be worth summarizing super shortly in the commit message

Updated.

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