tingwang added a comment.

(Inviting reviewers from D21611 <https://reviews.llvm.org/D21611> to look into 
the change here.)

The expected pattern `test8va()` added in this patch was broken by D21611 
<https://reviews.llvm.org/D21611>. Now as suggested by Ulrich, we are planning 
to create a flag for the original case. I have verified the functionality works 
with all kinds of aggregates smaller than SlotSize on PPC64. (If necessary, I 
can add those test case borrowed from 
clang/test/CodeGen/PowerPC/ppc-aggregate-abi.cpp) Let me know if you have any 
comments. Thank you!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D133338/new/

https://reviews.llvm.org/D133338

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to