[PATCH] D153898: [DebugInfo] Enable debug info emission for extern variables in C++

[David Blaikie via Phabricator via cfe-commits]

dblaikie added a comment.

Got some details on how much this costs? (eg: `bloaty` comparison for a clang bootstrap with/without this patch applied?)
Could you provide a quick summary of why this debug info is required? The definition should be provided in whatever translation unit defines the variable and you should be able to get debug info from there - if that translation unit is built without -g, then maybe this feature should be gated on `-fstandalone-debug` which is intended to express the intent to get debug info coverage when a translation unit may be the only one built with `-g`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153898