Reland [AArch64][DebugInfo] Do not recompute CalleeSavedStackSize (Take 2)
authorSander de Smalen <sander.desmalen@arm.com>
Tue, 29 Oct 2019 12:49:34 +0000 (12:49 +0000)
committerSander de Smalen <sander.desmalen@arm.com>
Tue, 29 Oct 2019 16:13:07 +0000 (16:13 +0000)
commitd6a7da80aae62474c15cad83d361830ffc4e2566
tree3c51ec8e41ad10e4a91fb1cbe1d4d1235e983e72
parentf228b5371647f471853c5fb3e6719823a42fe451
Reland [AArch64][DebugInfo] Do not recompute CalleeSavedStackSize (Take 2)

llvm/test/DebugInfo/MIR/X86/live-debug-values-reg-copy.mir failed with
EXPENSIVE_CHECKS enabled, causing the patch to be reverted in
rG2c496bb5309c972d59b11f05aee4782ddc087e71.

This patch relands the patch with a proper fix to the
live-debug-values-reg-copy.mir tests, by ensuring the MIR encodes the
callee-saves correctly so that the CalleeSaved info is taken from MIR
directly, rather than letting it be recalculated by the PEI pass. I've
done this by running `llc -stop-before=prologepilog` on the LLVM
IR as captured in the test files, adding the extra MOV instructions
that were manually added in the original test file, then running `llc
-run-pass=prologepilog` and finally re-added the comments for the MOV
instructions.
llvm/include/llvm/CodeGen/TargetFrameLowering.h
llvm/lib/CodeGen/LiveDebugValues.cpp
llvm/lib/CodeGen/RegUsageInfoCollector.cpp
llvm/lib/CodeGen/TargetFrameLoweringImpl.cpp
llvm/lib/Target/AArch64/AArch64FrameLowering.cpp
llvm/lib/Target/AArch64/AArch64MachineFunctionInfo.h
llvm/lib/Target/ARM/ARMFrameLowering.cpp
llvm/lib/Target/ARM/ARMFrameLowering.h
llvm/test/CodeGen/AArch64/wrong-callee-save-size-after-livedebugvariables.mir [new file with mode: 0644]
llvm/test/DebugInfo/MIR/Mips/live-debug-values-reg-copy.mir
llvm/test/DebugInfo/MIR/X86/live-debug-values-reg-copy.mir