[PATCH] Move estimateStackSize from ARM into MachineFunction
Hal Finkel
hfinkel at anl.gov
Thu Mar 14 12:45:02 PDT 2013
----- Original Message -----
> From: "Renato Golin" <renato.golin at linaro.org>
> To: "Tim Northover" <t.p.northover at gmail.com>
> Cc: "Hal Finkel" <hfinkel at anl.gov>, "LLVM Commits" <llvm-commits at cs.uiuc.edu>
> Sent: Thursday, March 14, 2013 2:38:20 PM
> Subject: Re: [PATCH] Move estimateStackSize from ARM into MachineFunction
>
>
> On 14 March 2013 19:24, Tim Northover < t.p.northover at gmail.com >
> wrote:
>
>
>
>
>
> It's a bit cheeky, but I don't suppose you could make AArch64 use the
> generic one too? Our implementation is identical to ARM's,
> byte-for-byte.
>
>
>
> Should be a trivial change, too.
>
>
> What's the complication of having it in MachineFrameInfo? Looks as a
> better place to put that in...
I don't think that there are any complications, I was just undecided. I'll put it into MFI (it is all the same source file anyway).
Thanks again,
Hal
>
>
>
> --renato
More information about the llvm-commits
mailing list