================
@@ -0,0 +1,258 @@
+//===- QualTypeMapperTest.cpp - Tests for QualType to ABI type mapping 
----===//
----------------
madhur13490 wrote:

Nice tests for pinning the type shapes. Since this is a 
classification-affecting change, though, I think we also want the usual two: an 
ABI unit test that calls computeInfo and checks the resulting ArgInfo 
(including a register-exhaustion case), and a dual-run parity test under 
clang/test/CodeGen/AArch64/ that runs with and without 
-fexperimental-abi-lowering against the same CHECK lines with 
--implicit-check-not="not yet implemented". Note that parity test can't pass 
while reportNYI fires — which is really the signal that these cases should be 
gated for now rather than tested around.

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

Reply via email to