Steven,

The assert has been in ToT for over a week now and I haven't seen any problems 
reported.
Is it time to move on to the next step?

Steve Ellcey
sell...@mips.com

________________________________________
From: Steven Bosscher [stevenb....@gmail.com]
Sent: Wednesday, May 29, 2013 3:15 PM
To: Steve Ellcey; gcc-patches@gcc.gnu.org; Andrew Bennett; 
rdsandif...@googlemail.com
Subject: Re: [patch, mips] Fix for PR target/56942


OK for trunk?

If it causes any trouble, please file a PR and assign it to me.

And when the dust has settled, I can clean up the FIXME for
JUMP_TABLE_DATA in next_active_insn, and fix up the back ends.

Ciao!
Steven


      * rtlanal.c (tablejump_p): Expect table and label to be adjacent.

Index: rtlanal.c
===================================================================
--- rtlanal.c   (revision 199324)
+++ rtlanal.c   (working copy)
@@ -2711,6 +2711,7 @@ tablejump_p (const_rtx insn, rtx *labelp, rtx *tab
       && (table = next_active_insn (label)) != NULL_RTX
       && JUMP_TABLE_DATA_P (table))
     {
+      gcc_assert (table == NEXT_INSN (label));
       if (labelp)
        *labelp = label;
       if (tablep)

Reply via email to