On Thu, 29 Aug 2019, Richard Biener wrote:

> On Thu, 29 Aug 2019, Jakub Jelinek wrote:
> 
> > On Thu, Aug 29, 2019 at 12:04:53PM +0200, Richard Biener wrote:
> > > + else
> > 
> > Perhaps use
> >     else if (MAY_HAVE_DEBUG_BIND_INSNS)
> > instead so that you don't walk it once again if there can't be DEBUG_INSNs?
> 
> Sure - will do as followup to unbreak bootstrap w/o re-testing this.

Bootstrapped / tested on x86_64-unknown-linux-gnu, applied.

Richard.

2019-08-29  Richard Biener  <rguent...@suse.de>

        * config/i386/i386-features.c (general_scalar_chain::convert_insn):
        Guard debug work with MAY_HAVE_DEBUG_BIND_INSNS.

Index: gcc/config/i386/i386-features.c
===================================================================
--- gcc/config/i386/i386-features.c     (revision 275030)
+++ gcc/config/i386/i386-features.c     (working copy)
@@ -893,7 +893,7 @@ general_scalar_chain::convert_insn (rtx_
        if (use)
          convert_reg (insn, DF_REF_REG (ref),
                       *defs_map.get (regno_reg_rtx [DF_REF_REGNO (ref)]));
-       else
+       else if (MAY_HAVE_DEBUG_BIND_INSNS)
          {
            /* If we generated a scalar copy we can leave debug-insns
               as-is, if not, we have to adjust them.  */

Reply via email to