[PATCH] D96045: [llvm-dwarfdump][locstats] Unify handling of inlined vars with no loc

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a comment.

In D96045#2707512 <https://reviews.llvm.org/D96045#2707512>, @djtodoro wrote:

> In D96045#2705750 <https://reviews.llvm.org/D96045#2705750>, @dblaikie wrote:
>
>> Oh, and it'd also be really good/important to make this work correctly with LTO (though that'll be more challenging/expensive due to the memory usage - it'll mean basically saving the details of every abstract origin (which means identifying them as such, since they don't expliictly identify themselves as abstract origins) (the expensive part) and then searching through that whenever we see a concrete-or-inlined (& if the search fails, putting the concrete-or-inlined in the list of things to resolve against future abstract origins that are parsed)). Will you be able to do that?
>
> Sure, I'll put it on my TODO list. I have some opened issues I am working on -- as soon as I am finished with it, I'll start working on that. The first step is to make a simple test case.

Thanks - happy to help with that any time. (roughly speaking - one translation unit with a function that's always_inline, and another translation unit that calls that first function - link them together with lto enabled and you'll get a cross-CU inlining)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96045