[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