[PATCH] D73534: [WIP][DebugInfo] Enable the debug entry values feature by default

[David Stenberg via Phabricator via llvm-commits]

dstenb added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineLICM.cpp:1370-1371
   // Otherwise we successfully unfolded a load that we can hoist.
+  if (MI->isCall())
+    MF.eraseCallSiteInfo(MI);
   MI->eraseFromParent();
----------------
Would it be possible to move these `eraseCallSiteInfo() / moveCallSiteInfo()` fixes to parent patch(es)? I guess that would mean that `-debug-entry-values` would have to be added to a number of tests where you saw those issues, just to be removed by this patch again, but I think it may overall make it easier to review the changes.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73534/new/

https://reviews.llvm.org/D73534