dblaikie added a comment.

In D70696#1765714 <https://reviews.llvm.org/D70696#1765714>, @yonghong-song 
wrote:

> @dblaikie Good points. I will guard external variable debug info generation 
> under `-fstandalone-debug` flag.


Oh, I figured given your needs this'd be guarded behind the target being BPF? 
While I don't mind it being also enabled by -fstandalone-debug (or, if you 
like, I guess maybe BPF is a "standalone-debug by default" target (like 
MacOS/Darwin, I think - due to some LLDB limitations at the moment) & maybe 
that captures all the BPF quirks in this regard?) - though I wouldn't leap to 
it unless someone else is already interested in that feature in 
-fstandalone-debug. (I'd be a bit worried about debug info growth and how "is 
this global variable referenced" is computed - eg: it could be referenced from 
a dead (uncalled) inline function)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70696



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to