summaryrefslogtreecommitdiffstats
path: root/clang/lib/CodeGen
diff options
context:
space:
mode:
authorChris Lattner <sabre@nondot.org>2011-02-28 07:16:14 +0000
committerChris Lattner <sabre@nondot.org>2011-02-28 07:16:14 +0000
commit62208c395a92cdef1a6a4fb242f7ea155abf142e (patch)
treefd5f0f2c518be7289eb740c9982920c4be5e01f8 /clang/lib/CodeGen
parent29dbbd12c17e8f87ba95dc9bdbf2cad7ed6e88d3 (diff)
downloadbcm5719-llvm-62208c395a92cdef1a6a4fb242f7ea155abf142e.tar.gz
bcm5719-llvm-62208c395a92cdef1a6a4fb242f7ea155abf142e.zip
make switch constant folding a bit stronger, handling a missed case.
llvm-svn: 126638
Diffstat (limited to 'clang/lib/CodeGen')
-rw-r--r--clang/lib/CodeGen/CGStmt.cpp11
1 files changed, 8 insertions, 3 deletions
diff --git a/clang/lib/CodeGen/CGStmt.cpp b/clang/lib/CodeGen/CGStmt.cpp
index c67d1d1b0da..21fb36a72d6 100644
--- a/clang/lib/CodeGen/CGStmt.cpp
+++ b/clang/lib/CodeGen/CGStmt.cpp
@@ -895,9 +895,14 @@ static CSFC_Result CollectStatementsForCase(const Stmt *S,
case CSFC_Success:
// A successful result means that either 1) that the statement doesn't
// have the case and is skippable, or 2) does contain the case value
- // and also contains the break to exit the switch. In either case,
- // we continue scanning the body of the compound statement to see if
- // the rest are skippable or have the case.
+ // and also contains the break to exit the switch. In the later case,
+ // we just verify the rest of the statements are elidable.
+ if (FoundCase) {
+ for (++I; I != E; ++I)
+ if (CodeGenFunction::ContainsLabel(*I, true))
+ return CSFC_Failure;
+ return CSFC_Success;
+ }
break;
case CSFC_FallThrough:
// If we have a fallthrough condition, then we must have found the
OpenPOWER on IntegriCloud