diff options
| author | Amir Ayupov <aaupov@fb.com> | 2023-01-03 17:31:44 -0800 |
|---|---|---|
| committer | Amir Ayupov <aaupov@fb.com> | 2023-01-03 17:32:11 -0800 |
| commit | f40d25dd8d3ad7bcfa8f5e8f74f245ab1a7675df (patch) | |
| tree | 0da96a6785f7e70e8fe9b2789fe53a76b533b5ad /bolt/include | |
| parent | bf4596bf5862c0a7092507463f6df9e80566a93d (diff) | |
| download | llvm-f40d25dd8d3ad7bcfa8f5e8f74f245ab1a7675df.tar.gz | |
[BOLT][NFC] Use llvm::reverse
Use llvm::reverse instead of `for (auto I = rbegin(), E = rend(); I != E; ++I)`
Reviewed By: #bolt, rafauler
Differential Revision: https://reviews.llvm.org/D140516
Diffstat (limited to 'bolt/include')
| -rw-r--r-- | bolt/include/bolt/Passes/DataflowAnalysis.h | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/bolt/include/bolt/Passes/DataflowAnalysis.h b/bolt/include/bolt/Passes/DataflowAnalysis.h index 71044434da45..2afaa6d3043a 100644 --- a/bolt/include/bolt/Passes/DataflowAnalysis.h +++ b/bolt/include/bolt/Passes/DataflowAnalysis.h @@ -341,12 +341,11 @@ public: } } } else { - for (auto I = Func.rbegin(), E = Func.rend(); I != E; ++I) { - Worklist.push(&*I); + for (BinaryBasicBlock &BB : llvm::reverse(Func)) { + Worklist.push(&BB); MCInst *Prev = nullptr; - for (auto J = (*I).rbegin(), E2 = (*I).rend(); J != E2; ++J) { - MCInst &Inst = *J; - PrevPoint[&Inst] = Prev ? ProgramPoint(Prev) : ProgramPoint(&*I); + for (MCInst &Inst : llvm::reverse(BB)) { + PrevPoint[&Inst] = Prev ? ProgramPoint(Prev) : ProgramPoint(&BB); Prev = &Inst; } } @@ -417,8 +416,8 @@ public: for (MCInst &Inst : *BB) doNext(Inst, *BB); else - for (auto I = BB->rbegin(), E = BB->rend(); I != E; ++I) - doNext(*I, *BB); + for (MCInst &Inst : llvm::reverse(*BB)) + doNext(Inst, *BB); if (Changed) { if (!Backward) { |