[PATCH] PR/34012, fwprop does not look at rtx_costs

[Richard Guenther]

On Nov 13, 2007 6:05 PM, Paolo Bonzini <paolo.bonzini@lu.unisi.ch> wrote:
>
> >> The referenced patch introduces:
> >>
> >> Running
> >> /home/uros/gcc-test-svn/trunk/gcc/testsuite/gcc.target/i386/i386.exp ...
> >> FAIL: gcc.target/i386/addr-sel-1.c scan-assembler a\\+1
> >> FAIL: gcc.target/i386/addr-sel-1.c scan-assembler b\\+1
> >>
> >> on 32bit x86 target.
>
> The attached patch, bootstrapped/regtested i686-pc-linux-gnu, fixes the
> failure.  Uros mentioned privately that the i386 part makes sense to him.

The middle-end changes also make sense to me.  So, this is ok if there's
no override from someone else in the next 24h.

Thanks,
Richard.

> Paolo
>
> 2007-11-12  Paolo Bonzini  <bonzini@gnu.org>
>
>         * rtlanal.c (rtx_cost): Don't change outer_code if we are
>         inside a MEM.
>         * config/i386/i386.c (ix86_rtx_costs): Return a cost of 0
>         for valid addresses involving PLUS and ASHIFT.