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

Wed Jun 28 17:09:51 PDT 2023

dblaikie accepted this revision.
dblaikie added a comment.

In D153898#4457263 <https://reviews.llvm.org/D153898#4457263>, @chiyuze wrote:

> Regarding cost, only BPF target triggers this debug info generation for extern variables.

Ah, OK - if that's what you need for BPF and it doesn't affect anything else - have at.

(though I'd still ask a bit about the issue of declaration V definition - is BPF just always "standalone" debug info generally? (is there a problem with the definition coming from another translation unit with debug info?)?)

> There is no impact on x86 builds of clang:

Oh, OK - no worries then.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153898