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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
