https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78516

--- Comment #15 from Vladimir Makarov <vmakarov at gcc dot gnu.org> ---
(In reply to Peter Bergner from comment #13)
> (In reply to Vladimir Makarov from comment #11)
> > Created attachment 40372 [details]
> > The proposed patch
> 
> Agreed your additions to my change looks good.  However, I'm not so sure
> about this last hunk:
> 
> -    SET_SRC (curr_insn_set) = new_reg;
> +    {
> +      SET_SRC (curr_insn_set) = new_reg;
> +      if (SUBREG_P (src))
> +     SUBREG_REG (SET_SRC (curr_insn_set)) = new_reg;
> +      else
> +     SET_SRC (curr_insn_set) = new_reg;
> +    }
> 
> Specifically, I don't think you meant to leave the following code as part of
> that:
> 
> +      SET_SRC (curr_insn_set) = new_reg;
> 
>

Sorry, I applied your changes manually and did a typo.  The line

SET_SRC (curr_insn_set) = new_reg;

should be removed.

I tested this patch with the typo on x86-64 and no new failures occurred. 
Strange.

Reply via email to