diff options
| author | David Blaikie <dblaikie@gmail.com> | 2012-04-09 16:29:35 +0000 |
|---|---|---|
| committer | David Blaikie <dblaikie@gmail.com> | 2012-04-09 16:29:35 +0000 |
| commit | 6bb5212b6e0ed2607f08217df7468cc4492f3197 (patch) | |
| tree | 236198af57690035134ce146e9228f98b83949e6 /include/llvm/Instructions.h | |
| parent | decbc43f72866fa33d18e4b63d0d2dd2a2e102f6 (diff) | |
| download | external_llvm-6bb5212b6e0ed2607f08217df7468cc4492f3197.zip external_llvm-6bb5212b6e0ed2607f08217df7468cc4492f3197.tar.gz external_llvm-6bb5212b6e0ed2607f08217df7468cc4492f3197.tar.bz2 | |
Fix accidentally constant conditions found by uncommitted improvements to -Wconstant-conversion.
A couple of cases where we were accidentally creating constant conditions by
something like "x == a || b" instead of "x == a || x == b". In one case a
conditional & then unreachable was used - I transformed this into a direct
assert instead.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@154324 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include/llvm/Instructions.h')
| -rw-r--r-- | include/llvm/Instructions.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/include/llvm/Instructions.h b/include/llvm/Instructions.h index e3676fe..f6eaf04 100644 --- a/include/llvm/Instructions.h +++ b/include/llvm/Instructions.h @@ -2507,7 +2507,8 @@ public: /// Resolves successor for current case. BasicBlockTy *getCaseSuccessor() { - assert((Index < SI->getNumCases() || DefaultPseudoIndex) && + assert((Index < SI->getNumCases() || + Index == DefaultPseudoIndex) && "Index out the number of cases."); return SI->getSuccessor(getSuccessorIndex()); } |