[PATCH] D71451: Support to emit extern variables debuginfo with "-fstandalone-debug"

[David Blaikie via Phabricator via cfe-commits]

dblaikie added a comment.

If this feature treats a declaration in a header different from one in a primary source file - I think that's problematic from a debug info/debugging perspective (& perhaps a quirk of how this functionality was originally implemented for the kernel feature thingy (the name I forget right now)... one I didn't notice & should probably go check up on to see if it makes sense there/how to rationalize it). I think it'd be surprising to users if moving a declaration into/out of a source file/header changed what they could do in a debugger and/or significantly increased debug info size, etc.


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

https://reviews.llvm.org/D71451