From: Andrew Pinski Date: Fri, 19 Aug 2005 05:41:13 +0000 (+0000) Subject: re PR c++/20624 (wrong "control reaches end of non-void function" warning) X-Git-Tag: misc/cutover-cvs2svn~1095 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=1581a9a177e82ba05a1769eebcf55a038569d6d3;p=thirdparty%2Fgcc.git re PR c++/20624 (wrong "control reaches end of non-void function" warning) 2005-08-19 Andrew Pinski PR middle-end/20624 * g++.dg/warn/Wreturn-3.C: New test 2005-08-19 Andrew Pinski PR middle-end/20624 * gimple-low.c (block_may_fallthru): Handle CLEANUP_POINT_EXPR by looking past it. From-SVN: r103268 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index a706bb2d9491..16ae2922c148 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2005-08-19 Andrew Pinski + + PR middle-end/20624 + * gimple-low.c (block_may_fallthru): Handle CLEANUP_POINT_EXPR by + looking past it. + 2005-08-18 Kaveh R. Ghazi * collect2.c (dup2): Delete. diff --git a/gcc/gimple-low.c b/gcc/gimple-low.c index 3bece1ac0d20..5c7c27f7e13f 100644 --- a/gcc/gimple-low.c +++ b/gcc/gimple-low.c @@ -368,6 +368,9 @@ block_may_fallthru (tree block) case CALL_EXPR: /* Functions that do not return do not fall through. */ return (call_expr_flags (stmt) & ECF_NORETURN) == 0; + + case CLEANUP_POINT_EXPR: + return block_may_fallthru (TREE_OPERAND (stmt, 0)); default: return true; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 1f0c092abb5e..c8dfcfb319c2 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2005-08-19 Andrew Pinski + + PR middle-end/20624 + * g++.dg/warn/Wreturn-3.C: New test + 2005-08-18 Dorit Nuzman PR tree-optimization/22228 diff --git a/gcc/testsuite/g++.dg/warn/Wreturn-3.C b/gcc/testsuite/g++.dg/warn/Wreturn-3.C new file mode 100644 index 000000000000..822dd4fdd3e9 --- /dev/null +++ b/gcc/testsuite/g++.dg/warn/Wreturn-3.C @@ -0,0 +1,18 @@ +// { dg-options "-Wreturn-type" } +// PR c++/20624 + +struct fpos { + fpos(int __pos) {} +}; +struct g { + g(); + ~g(); +}; +fpos seekoff(int b, int c) +{ + g __buf; + if (b != -1 && c >= 0) + return fpos(-1); + else + return fpos(-1); +}