@@ -572,11 +572,10 @@ class MemLocFragmentFill {
572
572
bool FirstMeet = true ;
573
573
// LiveIn locs for BB is the meet of the already-processed preds' LiveOut
574
574
// locs.
575
- for (auto I = pred_begin (&BB), E = pred_end (&BB); I != E; I++ ) {
575
+ for (const BasicBlock *Pred : predecessors (&BB)) {
576
576
// Ignore preds that haven't been processed yet. This is essentially the
577
577
// same as initialising all variables to implicit top value (⊤) which is
578
578
// the identity value for the meet operation.
579
- const BasicBlock *Pred = *I;
580
579
if (!Visited.count (Pred))
581
580
continue ;
582
581
@@ -941,10 +940,10 @@ class MemLocFragmentFill {
941
940
LLVM_DEBUG (dbgs () << BB->getName ()
942
941
<< " has new OutLocs, add succs to worklist: [ " );
943
942
LiveOut[BB] = std::move (LiveSet);
944
- for (auto I = succ_begin (BB), E = succ_end (BB); I != E; I++ ) {
945
- if (OnPending.insert (*I ).second ) {
946
- LLVM_DEBUG (dbgs () << I ->getName () << " " );
947
- Pending.push (BBToOrder[*I ]);
943
+ for (BasicBlock *Succ : successors (BB)) {
944
+ if (OnPending.insert (Succ ).second ) {
945
+ LLVM_DEBUG (dbgs () << Succ ->getName () << " " );
946
+ Pending.push (BBToOrder[Succ ]);
948
947
}
949
948
}
950
949
LLVM_DEBUG (dbgs () << " ]\n " );
@@ -2360,10 +2359,10 @@ bool AssignmentTrackingLowering::run(FunctionVarLocsBuilder *FnVarLocsBuilder) {
2360
2359
LLVM_DEBUG (dbgs () << BB->getName ()
2361
2360
<< " has new OutLocs, add succs to worklist: [ " );
2362
2361
LiveOut[BB] = std::move (LiveSet);
2363
- for (auto I = succ_begin (BB), E = succ_end (BB); I != E; I++ ) {
2364
- if (OnPending.insert (*I ).second ) {
2365
- LLVM_DEBUG (dbgs () << I ->getName () << " " );
2366
- Pending.push (BBToOrder[*I ]);
2362
+ for (BasicBlock *Succ : successors (BB)) {
2363
+ if (OnPending.insert (Succ ).second ) {
2364
+ LLVM_DEBUG (dbgs () << Succ ->getName () << " " );
2365
+ Pending.push (BBToOrder[Succ ]);
2367
2366
}
2368
2367
}
2369
2368
LLVM_DEBUG (dbgs () << " ]\n " );
0 commit comments