diff --git a/gcc/ChangeLog b/gcc/ChangeLog index e803c134535457d03e2a7d25737430efa2b74a01..a5309e1764986eaddc8121781fda0fea626e1fe6 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2010-05-14 Jason Merrill <jason@redhat.com> + + PR c++/44127 + * gimplify.c (gimplify_seq_add_stmt): No longer static. + * gimple.h: Declare it. + * gimple.c (gimple_build_eh_filter): No ops. + 2010-05-14 Jan Hubicka <jh@suse.cz> * ipa.c (enqueue_cgraph_node): Update comment; do not re-enqueue diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index a68f71c58f3ff216e3db32e4a9c8215207ebe8aa..eca1aaaa92c762450882262393e2fc4f38924f1a 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,9 @@ +2010-05-14 Jason Merrill <jason@redhat.com> + + PR c++/44127 + * cp-gimplify.c (gimplify_must_not_throw_expr): Use + gimple_build_eh_must_not_throw. + 2010-05-14 Martin Jambor <mjambor@suse.cz> * cp-lang.c (LANG_HOOKS_FOLD_OBJ_TYPE_REF): Remove both its undef diff --git a/gcc/cp/cp-gimplify.c b/gcc/cp/cp-gimplify.c index d6ae28fb97d62811683778c0245ba625845b3731..2e3f11d09466208243d82ee5795128f639084733 100644 --- a/gcc/cp/cp-gimplify.c +++ b/gcc/cp/cp-gimplify.c @@ -480,11 +480,16 @@ gimplify_must_not_throw_expr (tree *expr_p, gimple_seq *pre_p) tree stmt = *expr_p; tree temp = voidify_wrapper_expr (stmt, NULL); tree body = TREE_OPERAND (stmt, 0); + gimple_seq try_ = NULL; + gimple_seq catch_ = NULL; + gimple mnt; - stmt = build_gimple_eh_filter_tree (body, NULL_TREE, - build_call_n (terminate_node, 0)); + gimplify_and_add (body, &try_); + mnt = gimple_build_eh_must_not_throw (terminate_node); + gimplify_seq_add_stmt (&catch_, mnt); + mnt = gimple_build_try (try_, catch_, GIMPLE_TRY_CATCH); - gimplify_and_add (stmt, pre_p); + gimplify_seq_add_stmt (pre_p, mnt); if (temp) { *expr_p = temp; diff --git a/gcc/gimple.c b/gcc/gimple.c index 6f61ca7935a0ad251c2cd30709ee17b8c401beec..dd691a810d25acd4b41d81f2063dc8bad7b1f9fa 100644 --- a/gcc/gimple.c +++ b/gcc/gimple.c @@ -636,7 +636,7 @@ gimple_build_eh_filter (tree types, gimple_seq failure) gimple gimple_build_eh_must_not_throw (tree decl) { - gimple p = gimple_alloc (GIMPLE_EH_MUST_NOT_THROW, 1); + gimple p = gimple_alloc (GIMPLE_EH_MUST_NOT_THROW, 0); gcc_assert (TREE_CODE (decl) == FUNCTION_DECL); gcc_assert (flags_from_decl_or_type (decl) & ECF_NORETURN); diff --git a/gcc/gimple.h b/gcc/gimple.h index d1018b70c0aa3f38f8a55a4f74683021484f7055..8ecf7eb0a4edde50d95dc8c461e2ca906de0a2ed 100644 --- a/gcc/gimple.h +++ b/gcc/gimple.h @@ -221,6 +221,13 @@ gimple_seq_empty_p (const_gimple_seq s) void gimple_seq_add_stmt (gimple_seq *, gimple); +/* Link gimple statement GS to the end of the sequence *SEQ_P. If + *SEQ_P is NULL, a new sequence is allocated. This function is + similar to gimple_seq_add_stmt, but does not scan the operands. + During gimplification, we need to manipulate statement sequences + before the def/use vectors have been constructed. */ +void gimplify_seq_add_stmt (gimple_seq *, gimple); + /* Allocate a new sequence and initialize its first element with STMT. */ static inline gimple_seq diff --git a/gcc/gimplify.c b/gcc/gimplify.c index c74189194bd40a8ef3479224fae773634b1c1e1a..6d00fefe9ad453a55f9b1dbed3aa392ee3abed38 100644 --- a/gcc/gimplify.c +++ b/gcc/gimplify.c @@ -158,7 +158,7 @@ gimple_tree_eq (const void *p1, const void *p2) During gimplification, we need to manipulate statement sequences before the def/use vectors have been constructed. */ -static void +void gimplify_seq_add_stmt (gimple_seq *seq_p, gimple gs) { gimple_stmt_iterator si; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 92f955991be47bc46e21cf9bfbc524de7fce8cb0..331b142e024ee70e781a4aa5c5e099b2a54c9527 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2010-05-14 Jason Merrill <jason@redhat.com> + + PR c++/44127 + * g++.dg/eh/terminate1.C: New. + 2010-05-14 Jakub Jelinek <jakub@redhat.com> * gfortran.dg/gomp/pr44036-1.f90: Adjust. diff --git a/gcc/testsuite/g++.dg/eh/terminate1.C b/gcc/testsuite/g++.dg/eh/terminate1.C new file mode 100644 index 0000000000000000000000000000000000000000..cd60bccb48b9ba33f586d6b7967193d318b0bb42 --- /dev/null +++ b/gcc/testsuite/g++.dg/eh/terminate1.C @@ -0,0 +1,23 @@ +// PR c++/44127 + +// This is basically the same test as g++.eh/terminate1.C, but that one +// tests runtime behavior and this tests the assembly output. The test +// should call terminate (because initializing the catch parm throws), but +// from the personality routine, not directly. + +// { dg-final { scan-assembler-not "_ZSt9terminatev" } } + +struct A +{ + A() { } + A (const A&) { throw 1; } +}; + +int main() +{ + try + { + throw A(); + } + catch (A) { } +}