From 3576a595abffcd146439d99354a277ecb5e7e260 Mon Sep 17 00:00:00 2001 From: dougkwan Date: Fri, 25 Apr 2008 16:48:51 +0000 Subject: [PATCH] 2008-04-24 Doug Kwan * Makefile.in (STRICT_WARN): Remove -Wno-format and -Wno-missing-format-attribute. * gimplify.c (gimple_pop_condition): Remove redundant and incorrect gimple_seq_free. (gimplify_init_ctor_eval_range): Add a fall-through label for GIMPLE_COND statement. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/branches/gimple-tuples-branch@134670 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog.tuples | 9 +++++++++ gcc/Makefile.in | 4 +--- gcc/gimplify.c | 11 +++++++---- 3 files changed, 17 insertions(+), 7 deletions(-) diff --git a/gcc/ChangeLog.tuples b/gcc/ChangeLog.tuples index 6e632dd82f4..17f4089b436 100644 --- a/gcc/ChangeLog.tuples +++ b/gcc/ChangeLog.tuples @@ -1,3 +1,12 @@ +2008-04-24 Doug Kwan + + * Makefile.in (STRICT_WARN): Remove -Wno-format and + -Wno-missing-format-attribute. + * gimplify.c (gimple_pop_condition): Remove redundant and incorrect + gimple_seq_free. + (gimplify_init_ctor_eval_range): Add a fall-through label for + GIMPLE_COND statement. + 2008-04-25 Rafael Espindola * tree-ssa-dom.c (avail_expr_eq): Return false if the hashes don't diff --git a/gcc/Makefile.in b/gcc/Makefile.in index 7dd44d97e6f..be3e395619f 100644 --- a/gcc/Makefile.in +++ b/gcc/Makefile.in @@ -163,9 +163,7 @@ coverageexts = .{gcda,gcno} # CXX_COMPAT_WARN are C++ source compatibility warnings. LOOSE_WARN = @loose_warn@ # FIXME tuples. Do not merge this! -# We have to disable -Wformat and -Wmissing-format-attribute for bootstrapping. -STRICT_WARN = @strict_warn@ -Werror -Wno-return-type -Wno-format \ - -Wno-missing-format-attribute +STRICT_WARN = @strict_warn@ -Werror -Wno-return-type CXX_COMPAT_WARN = @cxx_compat_warn@ # This is set by --enable-checking. The idea is to catch forgotten diff --git a/gcc/gimplify.c b/gcc/gimplify.c index d583ca844e3..43f3f12ef43 100644 --- a/gcc/gimplify.c +++ b/gcc/gimplify.c @@ -298,7 +298,6 @@ gimple_pop_condition (gimple_seq *pre_p) if (conds == 0) { gimplify_seq_add_seq (pre_p, gimplify_ctxp->conditional_cleanups); - gimple_seq_free (gimplify_ctxp->conditional_cleanups); } } @@ -3137,11 +3136,12 @@ gimplify_init_ctor_eval_range (tree object, tree lower, tree upper, tree value, tree array_elt_type, gimple_seq *pre_p, bool cleared) { - tree loop_entry_label, loop_exit_label; + tree loop_entry_label, loop_exit_label, fall_thru_label; tree var, var_type, cref, tmp; loop_entry_label = create_artificial_label (); loop_exit_label = create_artificial_label (); + fall_thru_label = create_artificial_label (); /* Create and initialize the index variable. */ var_type = TREE_TYPE (upper); @@ -3167,8 +3167,11 @@ gimplify_init_ctor_eval_range (tree object, tree lower, tree upper, gimplify_seq_add_stmt (pre_p, gimple_build_assign (cref, value)); /* We exit the loop when the index var is equal to the upper bound. */ - gimplify_seq_add_stmt (pre_p, gimple_build_cond (EQ_EXPR, var, upper, - loop_exit_label, NULL_TREE)); + gimplify_seq_add_stmt (pre_p, + gimple_build_cond (EQ_EXPR, var, upper, + loop_exit_label, fall_thru_label)); + + gimplify_seq_add_stmt (pre_p, gimple_build_label (fall_thru_label)); /* Otherwise, increment the index var... */ tmp = build2 (PLUS_EXPR, var_type, var, -- 2.11.4.GIT