Re: [PATCH 3/4] aarch64: Tidy prologue local variables

2014-09-03 Thread Marcus Shawcroft
On 22 August 2014 23:05, Richard Henderson wrote: > Don't continually re-read data from cfun->machine. > > * config/aarch64/aarch64.c (aarch64_expand_prologue): Load > cfun->machine->frame.hard_fp_offset into a local variable. OK /Marcus

Re: [PATCH 3/4] aarch64: Tidy prologue local variables

2014-08-28 Thread Jiong Wang
On 28/08/14 17:48, Richard Henderson wrote: On 08/26/2014 05:58 AM, Jiong Wang wrote: there is a field "hardfp_offset" in aarch64_frame, and I think that field is not used and not initialized correctly. how about hoisting the calculation to aarch64_layout_frame to avoid duplicated calcuation h

Re: [PATCH 3/4] aarch64: Tidy prologue local variables

2014-08-28 Thread Richard Henderson
On 08/26/2014 05:58 AM, Jiong Wang wrote: > On 22/08/14 23:05, Richard Henderson wrote: >> Don't continually re-read data from cfun->machine. >> >> * config/aarch64/aarch64.c (aarch64_expand_prologue): Load >> cfun->machine->frame.hard_fp_offset into a local variable. >> --- >> gcc/config

Re: [PATCH 3/4] aarch64: Tidy prologue local variables

2014-08-26 Thread Jiong Wang
On 22/08/14 23:05, Richard Henderson wrote: Don't continually re-read data from cfun->machine. * config/aarch64/aarch64.c (aarch64_expand_prologue): Load cfun->machine->frame.hard_fp_offset into a local variable. --- gcc/config/aarch64/aarch64.c | 14 +++--- 1 file cha

[PATCH 3/4] aarch64: Tidy prologue local variables

2014-08-22 Thread Richard Henderson
Don't continually re-read data from cfun->machine. * config/aarch64/aarch64.c (aarch64_expand_prologue): Load cfun->machine->frame.hard_fp_offset into a local variable. --- gcc/config/aarch64/aarch64.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --gi