On 12-10-25 4:35 PM, Steven Bosscher wrote:
On Thu, Oct 25, 2012 at 10:24 PM, Jonathan Wakely <jwakely....@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.
