Re: Fix PR rtl-optimization/52629

2012-03-28 Thread Eric Botcazou
> What's the impact of this bug? If it is a wrong-code or ice-on-valid > bug then it's ok for the 4.7 branch. Neither, it's an out-of-bounds memory access in reload1.c with apparently no visible effects as it has been introduced with IRA (4.4 series). -- Eric Botcazou

Re: Fix PR rtl-optimization/52629

2012-03-28 Thread Richard Guenther
On Tue, 27 Mar 2012, Eric Botcazou wrote: > > Bootstrapped/regtested on x86_64-suse-linux, applied on mainline. Should > > it be applied to the release branches as well? > > > > > > 2012-03-26 Eric Botcazou > > > > PR rtl-optimization/52629 > > * reload1.c (count_pseudo): Short-circuit

Re: Fix PR rtl-optimization/52629

2012-03-27 Thread Eric Botcazou
> Bootstrapped/regtested on x86_64-suse-linux, applied on mainline. Should > it be applied to the release branches as well? > > > 2012-03-26 Eric Botcazou > > PR rtl-optimization/52629 > * reload1.c (count_pseudo): Short-circuit common case. > (count_spilled_pseudo): Return ea

Fix PR rtl-optimization/52629

2012-03-26 Thread Eric Botcazou
This is an out-of-bounds access in count_spilled_pseudo: we can enter the function for a pseudo without hard reg but we immediately use the hard reg number as an index: static void count_spilled_pseudo (int spilled, int spilled_nregs, int reg) { int freq = REG_FREQ (reg); int r = reg_renumbe