diff options
| author | Alexander Kornienko <alexfh@google.com> | 2015-12-28 13:06:58 +0000 |
|---|---|---|
| committer | Alexander Kornienko <alexfh@google.com> | 2015-12-28 13:06:58 +0000 |
| commit | 9c10490efe30ed75d52810c26e83189a96c04926 (patch) | |
| tree | 6abcb02d4f5e6aea72eca2429f4c5bf290a8ffc9 /clang/lib/StaticAnalyzer/Checkers/UnreachableCodeChecker.cpp | |
| parent | fba562004bf8a3c383453791bccbac3aaef7bcf0 (diff) | |
| download | bcm5719-llvm-9c10490efe30ed75d52810c26e83189a96c04926.tar.gz bcm5719-llvm-9c10490efe30ed75d52810c26e83189a96c04926.zip | |
Refactor: Simplify boolean conditional return statements in lib/StaticAnalyzer/Checkers
Summary: Use clang-tidy to simplify boolean conditional return values
Reviewers: dcoughlin, krememek
Subscribers: krememek, cfe-commits
Patch by Richard Thomson!
Differential Revision: http://reviews.llvm.org/D10021
llvm-svn: 256491
Diffstat (limited to 'clang/lib/StaticAnalyzer/Checkers/UnreachableCodeChecker.cpp')
| -rw-r--r-- | clang/lib/StaticAnalyzer/Checkers/UnreachableCodeChecker.cpp | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/clang/lib/StaticAnalyzer/Checkers/UnreachableCodeChecker.cpp b/clang/lib/StaticAnalyzer/Checkers/UnreachableCodeChecker.cpp index 9f66dc9be4c..a03abce9626 100644 --- a/clang/lib/StaticAnalyzer/Checkers/UnreachableCodeChecker.cpp +++ b/clang/lib/StaticAnalyzer/Checkers/UnreachableCodeChecker.cpp @@ -235,12 +235,9 @@ bool UnreachableCodeChecker::isInvalidPath(const CFGBlock *CB, return false; // Run each of the checks on the conditions - if (containsMacro(cond) || containsEnum(cond) - || containsStaticLocal(cond) || containsBuiltinOffsetOf(cond) - || containsStmt<UnaryExprOrTypeTraitExpr>(cond)) - return true; - - return false; + return containsMacro(cond) || containsEnum(cond) || + containsStaticLocal(cond) || containsBuiltinOffsetOf(cond) || + containsStmt<UnaryExprOrTypeTraitExpr>(cond); } // Returns true if the given CFGBlock is empty |
