Re: [2/7] v7: Handle frame pointer for stack alignment

["H.J. Lu" <hjl dot tools at gmail dot com>]

On Mon, Jun 30, 2008 at 10:18 AM, Ian Lance Taylor <iant@google.com> wrote:
> "H.J. Lu" <hjl.tools@gmail.com> writes:
>
>> The separate frame pointer initialization patch is at
>>
>>  http://gcc.gnu.org/ml/gcc-patches/2008-06/msg01269.html
>>
>> I have been testing it on Linux/ia32, Linux/x86-64 and Linux/ia64
>> without any stack alignment patches. OK for trunk?
>
> This patch is OK.
>

I checked in. Here is the updated 2/7 patch to handle frame
pointer from stack alignment

Frame pointer is always needed for stack alignment. If
is used either to save original stack pointer, we can
eliminate it.  But if dynamic realigned argument pointer
is needed, FP will be used to access local variable. In this
case, FP can't be eliminated.  OK for trunk with other
stack branches?

BTW, I can post a complete stack alignment patch
with a separate patch for testcases.

Thanks.


-- 
H.J.
---
2008-06-30  Joey Ye  <joey.ye@intel.com>
	    H.J. Lu  <hongjiu.lu@intel.com>

	* global.c (compute_regsets): Frame pointer is needed when
	stack is realigned.  Can eliminate frame pointer when stack is
	realigned and dynamic realigned argument pointer isn't used.
	* reload1.c (update_eliminables):  Frame pointer is needed
	when stack is realigned.
	(init_elim_table): Can eliminate frame pointer when stack is
	realigned and dynamic realigned argument pointer isn't used.

Attachment: gcc-stack-v8-2.txt
Description: Text document