This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: alpha bootstrap fix


> 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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]