diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-02-21 20:39:50 +0000 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-02-21 20:39:50 +0000 |
| commit | e9bc579c37c038abb66ac30489a1e26e5045ef9f (patch) | |
| tree | 3abf644028ed631aa020ef203627ddb7184677db /llvm/lib/CodeGen/SjLjEHPrepare.cpp | |
| parent | ec6f7fed54cdcc0e2588f912626314c0a58b24d7 (diff) | |
| download | bcm5719-llvm-e9bc579c37c038abb66ac30489a1e26e5045ef9f.tar.gz bcm5719-llvm-e9bc579c37c038abb66ac30489a1e26e5045ef9f.zip | |
ADT: Remove == and != comparisons between ilist iterators and pointers
I missed == and != when I removed implicit conversions between iterators
and pointers in r252380 since they were defined outside ilist_iterator.
Since they depend on getNodePtrUnchecked(), they indirectly rely on UB.
This commit removes all uses of these operators. (I'll delete the
operators themselves in a separate commit so that it can be easily
reverted if necessary.)
There should be NFC here.
llvm-svn: 261498
Diffstat (limited to 'llvm/lib/CodeGen/SjLjEHPrepare.cpp')
| -rw-r--r-- | llvm/lib/CodeGen/SjLjEHPrepare.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/CodeGen/SjLjEHPrepare.cpp b/llvm/lib/CodeGen/SjLjEHPrepare.cpp index daf32e4028b..ce01c5f23e5 100644 --- a/llvm/lib/CodeGen/SjLjEHPrepare.cpp +++ b/llvm/lib/CodeGen/SjLjEHPrepare.cpp @@ -419,7 +419,7 @@ bool SjLjEHPrepare::setupEntryBlockAndCallSites(Function &F) { // directly to the caller's context, which is what we want anyway, so no need // to do anything here. for (BasicBlock &BB : F) { - if (&BB == F.begin()) + if (&BB == &F.front()) continue; for (Instruction &I : BB) if (I.mayThrow()) @@ -434,7 +434,7 @@ bool SjLjEHPrepare::setupEntryBlockAndCallSites(Function &F) { // Following any allocas not in the entry block, update the saved SP in the // jmpbuf to the new value. for (BasicBlock &BB : F) { - if (&BB == F.begin()) + if (&BB == &F.front()) continue; for (Instruction &I : BB) { if (auto *CI = dyn_cast<CallInst>(&I)) { |

