[PATCH] D124998: [AArch64][SVE] Add aarch64_sve_pcs attribute to Clang

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 10 08:45:02 PDT 2022


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

In D124998#3503335 <https://reviews.llvm.org/D124998#3503335>, @paulwalker-arm wrote:

> @aaron.ballman It looks like the conversation reached a conclusion?  Given this is separate to what we're trying to add here can this patch be unblocked?

I continue to dislike adding more calling conventions, but the immediate concerns I had about running out of space are not too immediate and it sounds like we have a plausible way forward when we get closer to it. So I retract my blocking concerns -- this LGTM. Thanks for the discussion on this!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124998



More information about the cfe-commits mailing list