[llvm] r249852 - AArch64: Make getNextNode() cleanup in r249764 more clear

[Duncan P. N. Exon Smith via llvm-commits]

Author: dexonsmith
Date: Fri Oct  9 11:54:54 2015
New Revision: 249852

URL: http://llvm.org/viewvc/llvm-project?rev=249852&view=rev
Log:
AArch64: Make getNextNode() cleanup in r249764 more clear

After r249764, if you didn't see the full context, it looked like
`std::next(I)` would get the same result as
`++MachineBasicBlock::iterator(I)`.  However, `I` is a `MachineInstr*`
(not a `MachineBasicBlock::iterator`).

Use the `getIterator()` helper I added later (r249782) to make this code
more clear.

Modified:
    llvm/trunk/lib/Target/AArch64/AArch64CleanupLocalDynamicTLSPass.cpp

Modified: llvm/trunk/lib/Target/AArch64/AArch64CleanupLocalDynamicTLSPass.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64CleanupLocalDynamicTLSPass.cpp?rev=249852&r1=249851&r2=249852&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AArch64/AArch64CleanupLocalDynamicTLSPass.cpp (original)
+++ llvm/trunk/lib/Target/AArch64/AArch64CleanupLocalDynamicTLSPass.cpp Fri Oct  9 11:54:54 2015
@@ -118,8 +118,8 @@ struct LDTLSCleanup : public MachineFunc
 
     // Insert a copy from X0 to TLSBaseAddrReg for later.
     MachineInstr *Copy =
-        BuildMI(*I->getParent(), ++MachineBasicBlock::iterator(I),
-                I->getDebugLoc(), TII->get(TargetOpcode::COPY), *TLSBaseAddrReg)
+        BuildMI(*I->getParent(), ++I->getIterator(), I->getDebugLoc(),
+                TII->get(TargetOpcode::COPY), *TLSBaseAddrReg)
             .addReg(AArch64::X0);
 
     return Copy;