[PATCH] D21307: AArch4: Fix range loop contradicting comment above it

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 13 13:39:53 PDT 2016


arsenm created this revision.
arsenm added a subscriber: llvm-commits.

http://reviews.llvm.org/D21307

Files:
  lib/Target/AArch64/AArch64BranchRelaxation.cpp

Index: lib/Target/AArch64/AArch64BranchRelaxation.cpp
===================================================================
--- lib/Target/AArch64/AArch64BranchRelaxation.cpp
+++ lib/Target/AArch64/AArch64BranchRelaxation.cpp
@@ -463,7 +463,8 @@
   bool Changed = false;
   // Relaxing branches involves creating new basic blocks, so re-eval
   // end() for termination.
-  for (auto &MBB : *MF) {
+  for (MachineFunction::iterator I = MF->begin(); I != MF->end(); ++I) {
+    MachineBasicBlock &MBB = *I;
     MachineInstr *MI = MBB.getFirstTerminator();
     if (isConditionalBranch(MI->getOpcode()) &&
         !isBlockInRange(MI, getDestBlock(MI),


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21307.60598.patch
Type: text/x-patch
Size: 650 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160613/8d89e323/attachment.bin>


More information about the llvm-commits mailing list