From 94ca49167953b5f6445bdcdc6a44646e5eec4c23 Mon Sep 17 00:00:00 2001 From: law Date: Mon, 19 Mar 2007 19:52:19 +0000 Subject: [PATCH] * tree-cfg.c (find_taken_edge): Tighten conditions for optimizing computed gotos. * PR tree-optimization/30984 * gcc.c-torture/pr30984.c: New test. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@123067 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 5 +++++ gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/gcc.c-torture/compile/pr30984.c | 7 +++++++ gcc/tree-cfg.c | 13 ++++++++++++- 4 files changed, 29 insertions(+), 1 deletion(-) create mode 100644 gcc/testsuite/gcc.c-torture/compile/pr30984.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index fb839f5a7096..ec45c0e8364b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2007-03-19 Jeff Law + + * tree-cfg.c (find_taken_edge): Tighten conditions for + optimizing computed gotos. + 2007-03-19 Michael Matz * builtins.c (expand_builtin_sync_operation, diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index b62c6ebe1180..72474a89ce2b 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2007-03-19 Jeff Law + + * PR tree-optimization/30984 + * gcc.c-torture/pr30984.c: New test. + 2007-03-19 Andrew Pinski Richard Guenther diff --git a/gcc/testsuite/gcc.c-torture/compile/pr30984.c b/gcc/testsuite/gcc.c-torture/compile/pr30984.c new file mode 100644 index 000000000000..265a6f3616b1 --- /dev/null +++ b/gcc/testsuite/gcc.c-torture/compile/pr30984.c @@ -0,0 +1,7 @@ +int fs_exec(int ino) +{ + void *src = 0; + if (ino) + src = (void*)0xe000; + goto *src; +} diff --git a/gcc/tree-cfg.c b/gcc/tree-cfg.c index 202a69e2c8f0..fa4800e00c24 100644 --- a/gcc/tree-cfg.c +++ b/gcc/tree-cfg.c @@ -2039,7 +2039,18 @@ find_taken_edge (basic_block bb, tree val) return find_taken_edge_switch_expr (bb, val); if (computed_goto_p (stmt)) - return find_taken_edge_computed_goto (bb, TREE_OPERAND( val, 0)); + { + /* Only optimize if the argument is a label, if the argument is + not a label then we can not construct a proper CFG. + + It may be the case that we only need to allow the LABEL_REF to + appear inside an ADDR_EXPR, but we also allow the LABEL_REF to + appear inside a LABEL_EXPR just to be safe. */ + if ((TREE_CODE (val) == ADDR_EXPR || TREE_CODE (val) == LABEL_EXPR) + && TREE_CODE (TREE_OPERAND (val, 0)) == LABEL_DECL) + return find_taken_edge_computed_goto (bb, TREE_OPERAND (val, 0)); + return NULL; + } gcc_unreachable (); } -- 2.47.2