summaryrefslogtreecommitdiffstats
path: root/clang/lib/Analysis/UninitializedValues.cpp
diff options
context:
space:
mode:
authorTed Kremenek <kremenek@apple.com>2011-05-10 22:10:35 +0000
committerTed Kremenek <kremenek@apple.com>2011-05-10 22:10:35 +0000
commitefdb7fe53b14a3689a0b0fb50e5f0551aa606d15 (patch)
treee43cf9a7575177dce4cbf21877f4bc87baf6e484 /clang/lib/Analysis/UninitializedValues.cpp
parenta678098f246096c2dc1e392d2d25ad41daf55e1e (diff)
downloadbcm5719-llvm-efdb7fe53b14a3689a0b0fb50e5f0551aa606d15.tar.gz
bcm5719-llvm-efdb7fe53b14a3689a0b0fb50e5f0551aa606d15.zip
Fix crash in -Wuninitialized when using switch statments whose condition is a logical operation.
llvm-svn: 131158
Diffstat (limited to 'clang/lib/Analysis/UninitializedValues.cpp')
-rw-r--r--clang/lib/Analysis/UninitializedValues.cpp14
1 files changed, 9 insertions, 5 deletions
diff --git a/clang/lib/Analysis/UninitializedValues.cpp b/clang/lib/Analysis/UninitializedValues.cpp
index 88a2db751a4..fc0b904c83c 100644
--- a/clang/lib/Analysis/UninitializedValues.cpp
+++ b/clang/lib/Analysis/UninitializedValues.cpp
@@ -214,11 +214,15 @@ static BinaryOperator *getLogicalOperatorInChain(const CFGBlock *block) {
if (!b || !b->isLogicalOp())
return 0;
- if (block->pred_size() == 2 &&
- ((block->succ_size() == 2 && block->getTerminatorCondition() == b) ||
- block->size() == 1))
- return b;
-
+ if (block->pred_size() == 2) {
+ if (block->getTerminatorCondition() == b) {
+ if (block->succ_size() == 2)
+ return b;
+ }
+ else if (block->size() == 1)
+ return b;
+ }
+
return 0;
}
OpenPOWER on IntegriCloud