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

Eric Christopher echristo at gmail.com
Wed Jul 22 10:44:26 PDT 2015

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 :)


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


More information about the llvm-commits mailing list