Re: PATCH: Remove redundant instructions after regcprop

["Paulo J. Matos" <paulo at matos-sorge dot com>]

Thanks for the comments, I will be sending a new patch and fixed changelog.

On 02/08/12 11:27, Richard Guenther wrote:
> On Thu, Aug 2, 2012 at 12:19 PM, Paulo J. Matos <paulo@matos-sorge.com> wrote:
> > Forgot to mention: this is to fix PR 54154.
> >
> > Updated changelog:
> >
> > 2012-08-02   Paulo Matos <Paulo.Matos@csr.com>
> >
> >      PR middle-end/54154
> >
> >      * regcprop.c (copy_value): remove check for redundant moves.
> >      * regcprop.c (copy_value): add check for redundant moves,
> >      remove instructions if redundant.
>
> That's in copyprop_hardreg_forward_1
>
> +  gcc_assert(dr != sr);
> +
>
> space before ().
>
> +  FOR_BB_INSNS_SAFE(bb, insn, next)
>       {
>
> Likewise.
>
> +        unsigned int dr = REGNO(SET_DEST(set));
> +        unsigned int sr = REGNO(SET_SRC(set));
> +
>
> Likewise.
>
> Richard.
>
> >


--
PMatos