Re: [Patch]: Fix very large frame bug on i386

2012-05-25 Thread Tristan Gingold
On May 22, 2012, at 7:17 PM, Richard Henderson wrote: > On 05/15/12 02:10, Tristan Gingold wrote: >> 2012-05-15 Tristan Gingold >> >> * config/i386/i386.c (struct ix86_frame): Remove unused frame field. >> (ix86_compute_frame_layout): Fix type of stack_alignment_needed >> and p

Re: [Patch]: Fix very large frame bug on i386

2012-05-22 Thread Richard Henderson
On 05/15/12 02:10, Tristan Gingold wrote: > 2012-05-15 Tristan Gingold > > * config/i386/i386.c (struct ix86_frame): Remove unused frame field. > (ix86_compute_frame_layout): Fix type of stack_alignment_needed > and preferred_alignment. Ok with a test case. Even if it's a sc

Re: [Patch]: Fix very large frame bug on i386

2012-05-21 Thread Tristan Gingold
Ping for: On May 15, 2012, at 11:10 AM, Tristan Gingold wrote: > Hi, > > use of 'unsigned int' for i386.c:ix86_compute_frame_layout > stack_alignment_needed results in truncation of frame offset in code such as: > > offset = (offset + stack_alignment_needed - 1) & -stack_alignment_needed > >