Re: [PATCH] Remove gen_binary from combine.c (take 3)

[Mark Mitchell <mark at codesourcery dot com>]

Joseph S. Myers wrote:
> On Tue, 8 Mar 2005, Paolo Bonzini wrote:
>
>
> > 2005-03-01  Paolo Bonzini  <bonzini@gnu.org>
> >
> >        * combine.c (gen_binary): Remove.
> >        (known_cond, simplify_shift_const, find_split_point,
> >        combine_simplify_rtx, simplify_if_then_else, simplify_set,
> >        simplify_logical, expand_field_assignment, extract_left_shift,
> >        force_to_mode, if_then_else_cond, apply_distributive_law,
> >        simplify_and_const_int, simplify_shift_const, gen_lowpart_for_combine,
> >        simplify_comparison, reversed_comparison): Replace with
> >        simplify_gen_binary, simplify_gen_relational or
> >        distribute_and_simplify_rtx.
> >        (distribute_and_simplify_rtx): New function.
>
>
> This patch fixed bug 19454 (regression for bf64-1.c on ia64-hpux) on 
> mainline and when applied to 4.0 branch fixes the regression there as well 
> with no other changes to testsuite results, so I believe it should be 
> considered for 4.0 branch.

Agreed, but I'd like to see 4.0 test results on at least two other 
platforms, as this is rather heavy hammer.

--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304