[PATCH] D46599: [DbgInfo] Attempt to fix bug 37149

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added a comment.

So IIUC you are remembering the SlotIndex that the interval was trimmed to instead of assuming that it was the one immediately preceding the beginning to the interval?

I think it would help to add a testcase to illustrate what the situation that this is improving on.



================
Comment at: lib/CodeGen/LiveDebugVariables.cpp:166
   /// Set of interval start indexes that have been trimmed to the
   /// lexical scope.
+  SmallDenseMap<unsigned, SlotIndex, 2> trimmedDefs;
----------------
+ `, and the SlotIndex they were trimmed to.`?


Repository:
  rL LLVM

https://reviews.llvm.org/D46599