[PATCH] D71818: reland "[DebugInfo] Support to emit debugInfo for extern variables"

Yonghong Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 22 14:53:55 PST 2019


yonghong-song added a comment.

I cannot remove BPF target requirement as only BPF target can trigger debug info generation for extern variables. This is by design as discussed in D70696 <https://reviews.llvm.org/D70696> due to concerns it may blow up debug info binary size for existing applications. This may be relaxed in the future through some flags.
BTW, as far as I know, BPF is enabled for a lot of bots, ubuntu, x86, ppc, arm, etc. So the coverage of this future should be fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71818





More information about the cfe-commits mailing list