Make get_insns and friends inline

H.J. Lu hjl.tools@gmail.com
Tue Jun 1 21:07:00 GMT 2010


On Tue, Jun 1, 2010 at 1:36 PM, Jan Hubicka <hubicka@ucw.cz> wrote:
> Hi,
> this patch had a thinko in it that caused ICE when removing last instruction in sequence
> (that is seen in testsuite on some x86_64 configurations).
> Bootstrapped/regtested x86_64-linux and comitted as obvious.
>
> Honza
>
> Index: ChangeLog
> ===================================================================
> --- ChangeLog   (revision 160111)
> +++ ChangeLog   (working copy)
> @@ -1,5 +1,9 @@
>  2010-06-01  Jan Hubicka  <jh@suse.cz>
>
> +       * emit-rtl.c (remove_insn): Fix thinko in prevoius patch.
> +
> +2010-06-01  Jan Hubicka  <jh@suse.cz>
> +
>        * fwprop.c: Make emit-rtl.h include last.
>        * rtlanal.c: Include emit-rtl.h.
>        * genautomata.c: Output emit-rtl include into insn-automata.c
> Index: emit-rtl.c
> ===================================================================
> --- emit-rtl.c  (revision 160111)
> +++ emit-rtl.c  (working copy)
> @@ -3856,7 +3856,8 @@ remove_insn (rtx insn)
>     }
>   else if (get_insns () == insn)
>     {
> -      PREV_INSN (next) = NULL;
> +      if (next)
> +        PREV_INSN (next) = NULL;
>       set_first_insn (next);
>     }
>   else
>

Does it fix

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44370


-- 
H.J.



More information about the Gcc-patches mailing list