[PATCH] D79084: [VE] Dynamic stack allocation

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 18 18:26:29 PDT 2020


arsenm added a comment.

In D79084#2042936 <https://reviews.llvm.org/D79084#2042936>, @kaz7 wrote:

> In D79084#2024595 <https://reviews.llvm.org/D79084#2024595>, @simoll wrote:
>
> > - use standard C CC for grow_stack builtins.
>
>
> @simoll @arsenm
>
> Sorry for late reply.  I originally want to use it's own calling convention to allow caller to not spill registers since many program may call this function to grow their stack.  Is it not acceptable in general?  Or it's OK if there is a reason?
>
> It's OK to upstreaming C CC as a first step.  I would like to know what we should do to improve the performance of alloca or dynamic stack allocation in C++.  Thanks!


Would this just be the same as the existing PreserveMost or PreserveAll conventions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79084





More information about the llvm-commits mailing list