Message ID | CAFULd4asydptQzyOuyLWe673vxWN5oxUOU-3qpqFZayZ0Hj7Xw@mail.gmail.com |
---|---|
State | New |
Headers | show |
On Tue, Nov 13, 2012 at 06:07:20PM +0100, Uros Bizjak wrote: > @@ -242,7 +242,8 @@ create_pre_exit (int n_entities, int *entity_map, > int copy_start, copy_num; > int j; > > - if (INSN_P (return_copy)) > + if (INSN_P (return_copy) > + && !DEBUG_INSN_P (return_copy)) Please use if (NONDEBUG_INSN_P (return_copy)) instead. Jakub
On Tue, Nov 13, 2012 at 6:09 PM, Jakub Jelinek <jakub@redhat.com> wrote: >> int copy_start, copy_num; >> int j; >> >> - if (INSN_P (return_copy)) >> + if (INSN_P (return_copy) >> + && !DEBUG_INSN_P (return_copy)) > > Please use if (NONDEBUG_INSN_P (return_copy)) instead. Bah... I did look at this definition and for some reason unknown to me, I didn't see the equivalence. Anyway, I have committed the change. Thanks, Uros.
Index: mode-switching.c =================================================================== --- mode-switching.c (revision 193479) +++ mode-switching.c (working copy) @@ -242,7 +242,8 @@ create_pre_exit (int n_entities, int *entity_map, int copy_start, copy_num; int j; - if (INSN_P (return_copy)) + if (INSN_P (return_copy) + && !DEBUG_INSN_P (return_copy)) { /* When using SJLJ exceptions, the call to the unregister function is inserted between the @@ -330,7 +331,7 @@ create_pre_exit (int n_entities, int *entity_map, short_block = 1; break; } - if (copy_start >= FIRST_PSEUDO_REGISTER) + if (!targetm.calls.function_value_regno_p (copy_start)) { last_insn = return_copy; continue;