[Mlir-commits] [mlir] [mlir][emitc] mark `emitc.load` with `CExpression` (PR #130802)

Simon Camphausen llvmlistbot at llvm.org
Wed Mar 12 06:56:06 PDT 2025


simon-camp wrote:

Thanks for addressing this.

> Can you please add some tests for the output of translation? Interesting test case could also be loading from an array index, and then either having the emitc.subscript also within the `CExpression` or outside.

On top of that a translation test with a combination of a load inside an expression and a `call_opaque` op would be useful. I'm not sure if the precedence shouldn't be higher than that of any other op.

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


More information about the Mlir-commits mailing list