diff options
| author | Michael Gottesman <mgottesman@apple.com> | 2013-09-04 04:19:01 +0000 |
|---|---|---|
| committer | Michael Gottesman <mgottesman@apple.com> | 2013-09-04 04:19:01 +0000 |
| commit | 30955e9afb6548c79e426e5971e06bdfd3799ee0 (patch) | |
| tree | 98d21e1d5051c1a55820b297171fa0097665b955 | |
| parent | 06351cd0ff4146fa40ba9eb76bc10c5c6cb08616 (diff) | |
| download | external_llvm-30955e9afb6548c79e426e5971e06bdfd3799ee0.zip external_llvm-30955e9afb6548c79e426e5971e06bdfd3799ee0.tar.gz external_llvm-30955e9afb6548c79e426e5971e06bdfd3799ee0.tar.bz2 | |
Use llvm::next() instead of incrementing begin iterators of std::vector.
Iterator of std::vector may be implemented as a raw pointer. In
this case begin iterators are rvalues and cannot be incremented.
For example, this is the case with STDCXX implementation of vector.
Patch by Konstantin Tokarev <annulen@yandex.ru>.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@189911 91177308-0d34-0410-b5e6-96231b3b80d8
| -rw-r--r-- | lib/Target/R600/AMDILCFGStructurizer.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/Target/R600/AMDILCFGStructurizer.cpp b/lib/Target/R600/AMDILCFGStructurizer.cpp index 687eadb..481b5a7 100644 --- a/lib/Target/R600/AMDILCFGStructurizer.cpp +++ b/lib/Target/R600/AMDILCFGStructurizer.cpp @@ -1233,7 +1233,7 @@ int AMDGPUCFGStructurizer::handleJumpintoIfImp(MachineBasicBlock *HeadMBB, numClonedBlock += Num; Num += serialPatternMatch(*HeadMBB->succ_begin()); - Num += serialPatternMatch(*(++HeadMBB->succ_begin())); + Num += serialPatternMatch(*next(HeadMBB->succ_begin())); Num += ifPatternMatch(HeadMBB); assert(Num > 0); @@ -1713,7 +1713,7 @@ void AMDGPUCFGStructurizer::removeRedundantConditionalBranch( if (MBB->succ_size() != 2) return; MachineBasicBlock *MBB1 = *MBB->succ_begin(); - MachineBasicBlock *MBB2 = *(++MBB->succ_begin()); + MachineBasicBlock *MBB2 = *next(MBB->succ_begin()); if (MBB1 != MBB2) return; |