================
@@ -322,22 +322,27 @@ void CIRGenFunction::emitStoreOfScalar(mlir::Value value, 
Address addr,
   assert(!cir::MissingFeatures::opTBAA());
 }
 
+/// Helper method to check if the underlying ABI is AAPCS
+static bool isAAPCS(const TargetInfo &targetInfo) {
----------------
andykaylor wrote:

This seems like something we should be getting through a TargetInfo function 
call. I'm not asking you to change that in this PR, but can you add a TODO 
comment? This will be the fourth place in the code (two in CIR, two in classic 
codegen) that performs the check this way.

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

Reply via email to