[Mlir-commits] [mlir] [mlir][EmitC] Add support for pointer and opaque types to subscript op (PR #86266)

Matthias Gehre llvmlistbot at llvm.org
Fri Mar 22 06:30:26 PDT 2024


================
@@ -63,7 +63,8 @@ struct ConvertLoad final : public OpConversionPattern<memref::LoadOp> {
     }
 
     auto subscript = rewriter.create<emitc::SubscriptOp>(
-        op.getLoc(), operands.getMemref(), operands.getIndices());
+        op.getLoc(), cast<TypedValue<emitc::ArrayType>>(operands.getMemref()),
----------------
mgehre-amd wrote:

Can we allow the builder to take `Value` and leave the verification to the verifier? This is currently more verbose to write, and when it fails it would assert instead of being a verifier error.

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


More information about the Mlir-commits mailing list