[PATCH] D61940: [WIP][DebugInfo] Don't always extend variable locations when the reg location is unchanging

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added a reviewer: bjope.
bjope added a comment.

Adding myself as reviewer to remember to look at this patch.

Downstream we do things differently for our OOT target.  I tried to upstream that once-upon-a-time: https://reviews.llvm.org/D50638
But for some reason it never landed (I think I had some doubts about something).

We have also patched calculateDbgEntityHistory to step inside bundles. Not sure if we need that nowadays, but once upon a time we did not rely on that the def/use info on the BUNDLE instruction telling the full truth here (or maybe we allowed DEBUG_VALUE inside bundles... it is nothin upstream that prevents that afaik). So our downstream version here is slightly different compared to upstream for various historical reasons.

If we can get rid of the "ChangingRegs" stuff, then I guess we would get rid of some downstream hacks as well :-)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61940