[Mlir-commits] [mlir] [mlir][EmitC] Model lvalues as a type in EmitC (PR #91475)

Gil Rapaport llvmlistbot at llvm.org
Wed Jun 12 00:51:58 PDT 2024


================
@@ -835,6 +835,25 @@ def EmitC_LogicalOrOp : EmitC_BinaryOp<"logical_or", [CExpression]> {
   let assemblyFormat = "operands attr-dict `:` type(operands)";
 }
 
+def EmitC_LValueLoadOp : EmitC_Op<"lvalue_load", [
----------------
aniragil wrote:

Can we shorten to `LoadOp` and `emitc.load`? Encoding the required type in the name doesn't feel right.
Would also match `AssignOp` and `emitc.assign` which now also require an lvalue.

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


More information about the Mlir-commits mailing list