This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: alpha bootstrap fix
- To: Richard Henderson <rth at redhat dot com>, jh at suse dot cz, gcc-patches at gcc dot gnu dot org
- Subject: Re: alpha bootstrap fix
- From: Jan Hubicka <jh at suse dot cz>
- Date: Mon, 21 May 2001 11:18:01 +0200
- References: <20010520183219.A8134@redhat.com>
> Thu May 17 23:19:46 CEST 2001 Jan Hubicka <jh@suse.cz>
>
> * combine.c (combine_simplify_rtx): Use swap_commutative_operands_p.
> (gen_binary): Likewise.
> * optabs.c (emit_cmp_and_jump_insns, emit_conditional_move): Likewise.
> * simplify-rtx.c (simplify_gen_binary,
> simplify_gen_relational): Likewise.
>
> The change to emit_cmp_and_jump_insns, at minimum, is incorrect.
I apologize for the problem :(
> This is used e.g. by compare_from_rtx from emit_store_flag to
> generate setcc instructions. We have no way of mentioning this
This sounds interesting. I didn't added any new swapping code, just
made the old one possibly more aggresive. So the bug probably exist
in old code as well, but maybe setcc and friends does their
own swapping and then we probably should update them as well.
Honza