[PATCH] D133338: [clang][PowerPC] PPC64 VAArg use coerced integer type for direct aggregate fits in register

Ting Wang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 12 23:56:45 PDT 2022


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



More information about the cfe-commits mailing list