clayborg accepted this revision.
clayborg added a comment.
This revision is now accepted and ready to land.

In D110571#3033685 <https://reviews.llvm.org/D110571#3033685>, @jarin wrote:

> Cache only global variables.

I concur that we should only cache globals. So now we have unique variables for 
each block and they have the right symbol context? LGTM if so and if the test 
suite is happy. Thanks for digging into this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110571

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to