diff options
| author | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2010-09-19 21:21:10 +0000 |
|---|---|---|
| committer | Argyrios Kyrtzidis <akyrtzi@gmail.com> | 2010-09-19 21:21:10 +0000 |
| commit | 90963413104034cd883c93e08f7b30949429b169 (patch) | |
| tree | 044e423c1ecd45cc94b91f1b887c344fdc715e89 /clang/lib/AST/Expr.cpp | |
| parent | 7b8d2ae9129a68dfb058f209f2fec4db032639cf (diff) | |
| download | bcm5719-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.cpp | 6 |
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; |

