Re: New C++ regression: g++.dg/template/copy1.C

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

> 
> 
> This seems to be a new failing test; this one caused sometime in the
> last couple of days.  We're crashing in the gimplifier.
> 
> Does anyone know what patch caused this to fail?
Yes:
>        * c-gimplify.c (gimplify_decl_stmt): Update gimplify_type_sizes call.
>        Use gimplify_and_add.
>        * c-typeck.c (c_finish_if_stmt): Use NULL instead of empty stmt.
>        * gimplify.c (build_and_jump, gimplify_exit_expr,
>        gimplify_init_constructor, gimplify_save_expr, gimple_push_cleanup,
>        gimplify_stmt, gimplify_expr): Likewise.
>        (shortcut_cond_expr): Handle NULL arms of COND_EXPR.
>        (gimplify_statement_list): Remove NULL entries.
>        (gimplify_to_stmt_list): Handle NULL results.
>        (gimplify_type_sizes): Add list_p argument.
>        (gimplify_one_sizepos): Don't use internal pre/post queue.
>        * tree-gimple.h (gimplify_type_sizes): Update.

See <http://gcc.gnu.org/ml/gcc-patches/2004-06/msg01864.html>.




> 
> I'm serious considering asking the SC for permission to into a Stage 3
> mode for a little while here on the mainline to try to get things
> under control...

If that is the case I have been waiting for a patch to reviewed 
which fixes a regression which I caused for about a week now and
it causes some C++ testcases to fail at -O3 and nobody has commented
on it yet at all.

Patch here: <http://gcc.gnu.org/ml/gcc-patches/2004-06/msg01237.html>.

This patch also fixes a case dealing with IMA which Fariborz posted
a patch for the C-front-end's convert but since convert is no longer
called for the inliner, fold_convert is called instead we get an ICE
instead of an error.

There are other patches which have not been reviewed now from me which
I am less worried about and are not stage 3 material.

Thanks,
Andrew Pinski