[PATCH] D111916: [DebugInfo] Fixup DIEs for inlined functions

[Ellis Hoag via Phabricator via llvm-commits]

ellis added a comment.

In D111916#3067928 <https://reviews.llvm.org/D111916#3067928>, @dblaikie wrote:

> Maybe we could always create the abstract origin, then at the end collapse it down into the concrete definition if it's unused/there are no inline instances? Less need to try to separate things out?

Since some inlined functions also have concrete instances, we won't know until the end of the module if a function with a concrete instance should have an abstract origin or not. That means we have to maintain both an abstract origin and possibly a concrete DIE for every function, then at the end replace the abstract origin if it isn't used. I think this will cost a lot more memory and also require deleting a DIE, which I haven't seen in the code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111916