AArch64: Make getNextNode() cleanup in r249764 more clear
authorDuncan P. N. Exon Smith <dexonsmith@apple.com>
Fri, 9 Oct 2015 16:54:54 +0000 (16:54 +0000)
committerDuncan P. N. Exon Smith <dexonsmith@apple.com>
Fri, 9 Oct 2015 16:54:54 +0000 (16:54 +0000)
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.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@249852 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Target/AArch64/AArch64CleanupLocalDynamicTLSPass.cpp

index 2499b2c8875e9e68102d7bde7410d675638221bb..9310ac4a44a2d2bed3d617905e550ecbe1e99337 100644 (file)
@@ -118,8 +118,8 @@ struct LDTLSCleanup : public MachineFunctionPass {
 
     // 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;