Re: GCC 4.0 RC1 Available

[Andrew Pinski <pinskia at physics dot uc dot edu>]

On Apr 11, 2005, at 1:04 PM, Jakub Jelinek wrote:

> On Sun, Apr 10, 2005 at 03:05:17PM -0700, Mark Mitchell wrote:
> > The first GCC 4.0 candidate is available from:
> >
> > /pub/gcc/prerelease-4.0.0-20050410/
> >
> > on the usual gcc.gnu.org mirrors:
> >
> > http://gcc.gnu.org/mirrors.html
> >
> > I would like to know whether or not we have achieved the objective
> > aspects of the release criteria:
> >
> > http://gcc.gnu.org/gcc-4.0/criteria.html
> >
> > for primary and secondary platforms.  In particular, for primary 
> > platforms:
> >
> >     * The DejaGNU testsuite has been run, and compared with a run of
> > the testsuite on the previous release of GCC, and no regressions are
> > observed.
>
> Just an early warning.
> We are seeing (possible) miscompilation of KDE 3.4.0's 
> cssstyleselector.cpp
> on i386-linux, a regression between 20050330 (where it worked) and 
> 20050403.
> The resulting assembly with 20050403+ is also ~ 17% bigger.
> There are no differences in *.generic dump, but already in *.gimple
> dump the difference is huge.
> We'll keep looking into this.

This with optimization turned on right, then there is more inlining 
happening,
which was caused by:
2005-04-01  Richard Guenther <rguenth@tat.physik.uni-tuebingen.de>
            Jan Hubicka  <jh@suse.cz>
            Steven Bosscher <stevenb@suse.de>

        * cgraphunit.c (cgraph_estimate_size_after_inlining): Compute
        call cost based on argument sizes.
        (cgraph_mark_inline_edge): Avoid inline unit from shrinking by
        inlining.
        * params.def: (max-inline-inssn-single): Set to 450.
        (max-inline-insns-auto): Set to 90.
        (max-inline-insns-recursive): Set to 450
        (max-inline-insns-recursive-auto): Set to 450.
        (large-function-insns): Set to 2700.
        (inline-call-cost): New parameter.
        * tree-inline.c (estimate_move_cost): New function.
        (estimate_num_insns_1): Compute move sizes costs by 
estimate_move_cost
        for non-gimple-regs, set cost to 0 for gimple-regs.  Compute 
call size
        based on arguments.
        * tree-inline.h (estimate_move_cost): Declare.
        * invoke.texi: (max-inline-inssn-single): Change default to 450.
        (max-inline-insns-auto): Change default to 90.
        (max-inline-insns-recursive): Change default to 450
        (max-inline-insns-recursive-auto): Change default to 450.
        (large-function-insns): Change default to 2700.
        (inline-call-cost): Document new parameter.

-- Pinski