[PATCH] D107076: [DWARF] Revert sharing subprograms across CUs

[David Blaikie via Phabricator via llvm-commits]

dblaikie added a subscriber: dexonsmith.
dblaikie added a comment.

I'm generally in favor of this direction forward - I don't think it should fall to you to fix the debug info and if it isn't at the top of @vsk's list for a bit, let's fallback to known good-ish state for now.

@vsk - you might want to talk to @dexonsmith and @aprantl about the IR representation of these extra DISubprogram declarations that're being used for call site metadata, their lack of appropriate deduplication I think means you/we should consider some significant representational changes/fixes to make this work more reliably/consistently. See https://reviews.llvm.org/D94976#2508262 for more details.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107076