commit 4ead40c1e945e23b82f7fc725c22df50c5756a78
author Tim Northover <tnorthover@apple.com> Thu Mar 09 21:12:06 2017 +0000
committer Copybara-Service <copybara-worker@google.com> Sun Oct 11 21:31:23 2020 -0700
tree a9dfba7132a2cfe7afe4638aebf5d3f61102021b
parent ec395ae6f93ada6f572d37c633dff212d1258340

GlobalISel: put debug info for static allocas in the MachineFunction.

The good reason to do this is that static allocas are pretty simple to handle
(especially at -O0) and avoiding tracking DBG_VALUEs throughout the pipeline
should give some kind of performance benefit.

The bad reason is that the debug pipeline is an unholy mess of implicit
contracts, where determining whether "DBG_VALUE %reg, imm" actually implies a
load or not involves the services of at least 3 soothsayers and the sacrifice
of at least one chicken.  And it still gets it wrong if the variable is at SP
directly.

llvm-svn: 297410
GitOrigin-RevId: 7a9ea8f628f863c2cde41e63be45a2e47fbd95ef

lib/CodeGen/MachineFunction.cpp

diff --git a/lib/CodeGen/MachineFunction.cpp b/lib/CodeGen/MachineFunction.cpp
index 831e7d8..ef5e032 100644
--- a/lib/CodeGen/MachineFunction.cpp
+++ b/lib/CodeGen/MachineFunction.cpp
@@ -169,6 +169,7 @@
   InstructionRecycler.clear(Allocator);
   OperandRecycler.clear(Allocator);
   BasicBlockRecycler.clear(Allocator);
+  VariableDbgInfos.clear();
   if (RegInfo) {
     RegInfo->~MachineRegisterInfo();
     Allocator.Deallocate(RegInfo);