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

Simon Camphausen llvmlistbot at llvm.org
Thu Jun 13 01:24:59 PDT 2024


================
@@ -61,6 +61,9 @@ void mlir::emitc::buildTerminatedBody(OpBuilder &builder, Location loc) {
 bool mlir::emitc::isSupportedEmitCType(Type type) {
   if (llvm::isa<emitc::OpaqueType>(type))
     return true;
+  if (auto lType = llvm::dyn_cast<emitc::LValueType>(type))
+    // lvalue types are only allowed in a few places.
+    return false;
----------------
simon-camp wrote:

I don't have a strong opinion on this, but I thought it wouldn't hurt to explicitly write this out.

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


More information about the Mlir-commits mailing list