summaryrefslogtreecommitdiffstats
path: root/clang/lib/AST/Expr.cpp
diff options
context:
space:
mode:
authorArgyrios Kyrtzidis <akyrtzi@gmail.com>2010-09-19 21:21:10 +0000
committerArgyrios Kyrtzidis <akyrtzi@gmail.com>2010-09-19 21:21:10 +0000
commit90963413104034cd883c93e08f7b30949429b169 (patch)
tree044e423c1ecd45cc94b91f1b887c344fdc715e89 /clang/lib/AST/Expr.cpp
parent7b8d2ae9129a68dfb058f209f2fec4db032639cf (diff)
downloadbcm5719-llvm-90963413104034cd883c93e08f7b30949429b169.tar.gz
bcm5719-llvm-90963413104034cd883c93e08f7b30949429b169.zip
Warn when an expression result in a LabelStmt is unused.
llvm-svn: 114314
Diffstat (limited to 'clang/lib/AST/Expr.cpp')
-rw-r--r--clang/lib/AST/Expr.cpp6
1 files changed, 5 insertions, 1 deletions
diff --git a/clang/lib/AST/Expr.cpp b/clang/lib/AST/Expr.cpp
index 4c6822c041f..b31ec2314dc 100644
--- a/clang/lib/AST/Expr.cpp
+++ b/clang/lib/AST/Expr.cpp
@@ -1225,9 +1225,13 @@ bool Expr::isUnusedResultAWarning(SourceLocation &Loc, SourceRange &R1,
// however, if the result of the stmt expr is dead, we don't want to emit a
// warning.
const CompoundStmt *CS = cast<StmtExpr>(this)->getSubStmt();
- if (!CS->body_empty())
+ if (!CS->body_empty()) {
if (const Expr *E = dyn_cast<Expr>(CS->body_back()))
return E->isUnusedResultAWarning(Loc, R1, R2, Ctx);
+ if (const LabelStmt *Label = dyn_cast<LabelStmt>(CS->body_back()))
+ if (const Expr *E = dyn_cast<Expr>(Label->getSubStmt()))
+ return E->isUnusedResultAWarning(Loc, R1, R2, Ctx);
+ }
if (getType()->isVoidType())
return false;
OpenPOWER on IntegriCloud