Message ID | CABu31nNgwXbXSPhKcODa+LgDw2BqXDT7RzdVO4H46RA9=DcQ6g@mail.gmail.com |
---|---|
State | New |
Headers | show |
On 12-10-25 4:35 PM, Steven Bosscher wrote: > On Thu, Oct 25, 2012 at 10:24 PM, Jonathan Wakely <jwakely.gcc@gmail.com> wrote: >> This seems to fix it, is it correct? (Untested as I'm still waiting >> for a bootstrap to finish) > I'd do it the other way around: > > --- lra-constraints.c 2012-10-24 13:39:19.830019609 -0700 > +++ lra-constraints.c 2012-10-25 13:32:39.990019608 -0700 > @@ -1204,10 +1204,8 @@ > || ((secondary_class != NO_REGS || sri.icode != CODE_FOR_nothing) > && dclass != NO_REGS)) > { > -#if ENABLE_ASSERT_CHECKING > enum reg_class old_sclass = secondary_class; > secondary_reload_info old_sri = sri; > -#endif > > sri.prev_sri = NULL; > sri.icode = CODE_FOR_nothing; > > > lra_assert is the same as gcc_checking_assert already so the whole > thing disappears with release checking. > > (lra_assert and ira_assert should just be replaced with > gcc_checking_assert, really..) > > Thanks, Steven. I committed your recommended patch for LRA.
--- lra-constraints.c 2012-10-24 13:39:19.830019609 -0700 +++ lra-constraints.c 2012-10-25 13:32:39.990019608 -0700 @@ -1204,10 +1204,8 @@ || ((secondary_class != NO_REGS || sri.icode != CODE_FOR_nothing) && dclass != NO_REGS)) { -#if ENABLE_ASSERT_CHECKING enum reg_class old_sclass = secondary_class; secondary_reload_info old_sri = sri; -#endif sri.prev_sri = NULL; sri.icode = CODE_FOR_nothing;