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

[Markus Böck]

zero9178 wrote:

> Do you suggest that the string tags should be kept separately?

I believe this should result in less code and things that could go out of sync. Keeping the number of operands in the current `Variadic` in sync with the indices used in the operand bundles rather difficult and the implementation detail of the ranges being denoted via indices even leaks into the assembly format. 

With the `VariadicOfVariadic` the only thing that needs to be kept in sync is the number of variadic ranges and the number of corresponding string tags. We could then write a custom printer and parser that prints each operand range together with its string tag.



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