@@ -572,11 +572,10 @@ class MemLocFragmentFill {
572572 bool FirstMeet = true ;
573573 // LiveIn locs for BB is the meet of the already-processed preds' LiveOut
574574 // locs.
575- for (auto I = pred_begin (&BB), E = pred_end (&BB); I != E; I++ ) {
575+ for (const BasicBlock *Pred : predecessors (&BB)) {
576576 // Ignore preds that haven't been processed yet. This is essentially the
577577 // same as initialising all variables to implicit top value (⊤) which is
578578 // the identity value for the meet operation.
579- const BasicBlock *Pred = *I;
580579 if (!Visited.count (Pred))
581580 continue ;
582581
@@ -941,10 +940,10 @@ class MemLocFragmentFill {
941940 LLVM_DEBUG (dbgs () << BB->getName ()
942941 << " has new OutLocs, add succs to worklist: [ " );
943942 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 ]);
948947 }
949948 }
950949 LLVM_DEBUG (dbgs () << " ]\n " );
@@ -2360,10 +2359,10 @@ bool AssignmentTrackingLowering::run(FunctionVarLocsBuilder *FnVarLocsBuilder) {
23602359 LLVM_DEBUG (dbgs () << BB->getName ()
23612360 << " has new OutLocs, add succs to worklist: [ " );
23622361 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 ]);
23672366 }
23682367 }
23692368 LLVM_DEBUG (dbgs () << " ]\n " );
0 commit comments