================
@@ -410,6 +407,27 @@ void insertReturnCoercion(mlir::FunctionOpInterface funcOp,
   }
 }
 
+/// A whole-record value's backing storage: the plain load that produced it and
----------------
erichkeane wrote:

This comment is really awkward and in a really strange voice, it should be 
re-done to better explain what this is.

Also, why not just a `std::pair`?  A special type for this seems 
odd/unnecessary? 

https://github.com/llvm/llvm-project/pull/216499
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to