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

2023-06-28 Thread Yuze Chi via Phabricator via cfe-commits
chiyuze added a comment. Regarding cost, only BPF target triggers this debug info generation for extern variables. There is no impact on x86 builds of clang: Without patch: bloaty build/RelWithDebInfo/83d47ba15a1229a21aaca8a8d6a33e0e90aabfd4/bin/clang FILE SIZEVM SIZE

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

2023-06-29 Thread Yuze Chi via Phabricator via cfe-commits
chiyuze added a comment. Thank you for accepting this patch! In D153898#4457432 , @MaskRay wrote: > I wonder whether you have ready-to-use instructions to test this for folks > who are not familiar with Linux kernel/eBPF. (I happened to start to read

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

2023-06-27 Thread Yuze Chi via Phabricator via cfe-commits
chiyuze created this revision. chiyuze added a reviewer: rnk. Herald added a project: All. chiyuze requested review of this revision. Herald added a project: clang. Debug info emission for extern variables in C++ was previously disabled when the functionality was added in