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

[Tobias Gysi]

https://github.com/gysit commented:

Thanks for working on this!

I added some mostly minor comments. It seems like some verifier tests and probably also some roundtrip.mlir tests for printing and parsing could make sense?

If you have time to add support for the llvm.intr.assume that would be great.

I think mid term we also want to support operand bundles when importing from LLVM IR into MLIR LLVM dialect (ModuleImport.cpp). However, given the size of the PR it may make sense to do that in a later stage.

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