diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-04-20 18:27:18 +0000 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-04-20 18:27:18 +0000 |
| commit | 8ec8da40a860b7e0dca0b6fe7d4b1e6821630f01 (patch) | |
| tree | 80c2e18267b490d1f951e5db000714884d2d3e62 /llvm/lib/IR/Verifier.cpp | |
| parent | a479b968e91ec65de016e49d2b7ffef1ae035370 (diff) | |
| download | bcm5719-llvm-8ec8da40a860b7e0dca0b6fe7d4b1e6821630f01.tar.gz bcm5719-llvm-8ec8da40a860b7e0dca0b6fe7d4b1e6821630f01.zip | |
Verifier: Prefer early continue over if-nesting, NFC
llvm-svn: 266897
Diffstat (limited to 'llvm/lib/IR/Verifier.cpp')
| -rw-r--r-- | llvm/lib/IR/Verifier.cpp | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/llvm/lib/IR/Verifier.cpp b/llvm/lib/IR/Verifier.cpp index f010fd35dea..e643dff324c 100644 --- a/llvm/lib/IR/Verifier.cpp +++ b/llvm/lib/IR/Verifier.cpp @@ -258,15 +258,16 @@ public: return false; } for (const BasicBlock &BB : F) { - if (BB.empty() || !BB.back().isTerminator()) { - if (OS) { - *OS << "Basic Block in function '" << F.getName() - << "' does not have terminator!\n"; - BB.printAsOperand(*OS, true); - *OS << "\n"; - } - return false; + if (!BB.empty() && BB.back().isTerminator()) + continue; + + if (OS) { + *OS << "Basic Block in function '" << F.getName() + << "' does not have terminator!\n"; + BB.printAsOperand(*OS, true); + *OS << "\n"; } + return false; } // Now directly compute a dominance tree. We don't rely on the pass |
