[PATCH] D11268: [DebugInfo] Fix ARM/AArch64 prologue_end position

[Eric Christopher]

echristo added a subscriber: echristo.
echristo added a comment.

One inline comment. My general preference would be an explicit DebugLoc in all of these uses, but maybe it doesn't matter that much. The inline comment does :)

-eric


================
Comment at: lib/Target/ARM/ARMFrameLowering.cpp:331-332
@@ -326,4 +330,4 @@
   if (ArgRegsSaveSize) {
     emitSPUpdate(isARM, MBB, MBBI, dl, TII, -ArgRegsSaveSize,
                  MachineInstr::FrameSetup);
     DefCFAOffsetCandidates.addInst(std::prev(MBBI), ArgRegsSaveSize, true);
----------------
The DebugLoc for all uses of emitSPUpdate is now just DebugLoc so you can remove the parameter.


http://reviews.llvm.org/D11268