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

Sirui Mu llvmlistbot at llvm.org
Tue Sep 17 06:57:08 PDT 2024


Lancern wrote:

> Is there a reason we cannot use [VariadicOfVariadic](https://github.com/llvm/llvm-project/blob/019136e30fea8b8e6ef9c6dda1f9dd20a504c573/mlir/include/mlir/IR/CommonTypeConstraints.td#L135) for bundle_operands?
op_bundles would then need to be a DenseI32ArrayAttr, but ought to be kept up to sync automatically by all APIs I think.

Each operand bundle has a string tag associated with it, thus I have to invent the `#llvm.opbundle` stuff to store this tag. Do you suggest that the string tags should be kept separately?

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


More information about the Mlir-commits mailing list