[Mlir-commits] [mlir] [MLIR][LLVMIR] llvm.call_intrinsic: support operand/result attributes (PR #129640)

[Bruno Cardoso Lopes]

bcardosolopes wrote:

> Does this still trigger the bind twice problem?

Updated one comment with more description. The problem is the actual lack of `OperationState` being available to pass in to both `addArgAndResultAttrs` and `parseCallTypeAndResolveOperands`. I think we might be able to keep the `assemblyFormat` if strip some of the guts from both into smaller ones and use them into the custom parse/print function. Maybe it's possible to add those smaller things to `call_interface_impl`, not sure. If we wanna keep the `assemblyFormat` that might be the tradeoff (it might not work also, let's see). Wdyt?

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