On Wed, 2008-05-07 at 07:45 -0700, Steve Ellcey wrote:
> I have found that this problem does not occur on the ToT sources and
> that the problem went away with this patch:
> 
>   2008-04-07  Peter Bergner  <[EMAIL PROTECTED]>
> 
>        PR middle-end/PR28690
>        * rtlanal.c: Update copyright years.
>        (commutative_operand_precedence): Give SYMBOL_REF's the same precedence
>        as REG_POINTER and MEM_POINTER operands.
>        * emit-rtl.c (gen_reg_rtx_and_attrs): New function.
>        (set_reg_attrs_from_value): Call mark_reg_pointer as appropriate.
>        * rtl.h (gen_reg_rtx_and_attrs): Add prototype for new function.
>        * gcse.c: Update copyright years.
>        (pre_delete): Call gen_reg_rtx_and_attrs.
>        (hoist_code): Likewise.
>        (build_store_vectors): Likewise.
>        (delete_store): Likewise.
>        * loop-invariant.c (move_invariant_reg): Likewise.
>        Update copyright years.
> 
> I don't know if porting this patch to the 4.3 branch is an option or not
> but it might be the easiest way to fix this problem without having to
> revert Andrew's patch.

Note that the rtlanal.c:commutative_operand_precedence() hunk was reverted
because it caused some problems on CRIS and was replaced by the following
safer change:

    http://gcc.gnu.org/ml/gcc-patches/2008-04/msg00693.html

Peter



Reply via email to