Fix find_moveable_pseudos, PR52997

Ulrich Weigand uweigand@de.ibm.com
Wed May 2 20:59:00 GMT 2012


Bernd Schmidt wrote:
> On 04/27/2012 06:25 PM, Ulrich Weigand wrote:
> > Bernd Schmidt wrote:
> >
> >> We're creating new pseudos, and while we're resizing some data
> >> structures, we aren't doing it for everything.
> >
> >> @@ -3983,7 +3983,8 @@ find_moveable_pseudos (void)
> >>
> >>     last_moveable_pseudo = max_reg_num ();
> >>
> >> -  fix_reg_equiv_init();
> >> +  fix_reg_equiv_init ();
> >> +  resize_reg_info ();
> >>     regstat_free_n_sets_and_refs ();
> >>     regstat_free_ri ();
> >>     regstat_init_n_sets_and_refs ();
> >
> > This causes a bootstrap failure on s390 when enabling
> > -fsched-pressure -fsched-pressure-algorithm=model by default
> > (not sure why this doesn't show up without that change).
> >
> > The problem is that resize_reg_info only resizes the data
> > structure, but leaves it uninitialized.
> 
> Argh. Something like this maybe (currently testing on i686-linux, ok if 
> it passes?)

This has fixed the problem I was seeing, thanks!

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com



More information about the Gcc-patches mailing list