Skip to content

Commit 42e7796

Browse files
committed
[ARM] Add a comment about fixupImmediateBr updaing ImmBranches. NFC
To prevent people from modernizing the loop, add a comment that fixupImmediateBr can append to ImmBranches.
1 parent bab9d4c commit 42e7796

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

llvm/lib/Target/ARM/ARMConstantIslandPass.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -476,8 +476,10 @@ bool ARMConstantIslands::runOnMachineFunction(MachineFunction &mf) {
476476

477477
LLVM_DEBUG(dbgs() << "Beginning BR iteration #" << NoBRIters << '\n');
478478
bool BRChange = false;
479-
for (unsigned i = 0, e = ImmBranches.size(); i != e; ++i)
479+
for (unsigned i = 0, e = ImmBranches.size(); i != e; ++i) {
480+
// Note: fixupImmediateBr can append to ImmBranches.
480481
BRChange |= fixupImmediateBr(ImmBranches[i]);
482+
}
481483
if (BRChange && ++NoBRIters > 30)
482484
report_fatal_error("Branch Fix Up pass failed to converge!");
483485
LLVM_DEBUG(dumpBBs());

0 commit comments

Comments
 (0)