[PATCH] D149058: [BPF][DebugInfo] Use .BPF.ext for line info when DWARF is not available
Yonghong Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 8 06:48:50 PDT 2023
yonghong-song added a comment.
> Sorry, not sure I understand. The only way to preserve old interface for `DILineInfo` that I see is to collect all lines stored in .BTF.ext, sort those lines by file name, line number and create `std::string` instances with "fake" source code for each file (which is a bit wasteful from cpu/memory pov). Is that what you mean or am I missing something?
Okay, I see. I thought we could add 'LineSource' to BTFContext, but that will need to change the generic llvm-objdump interface (casting to BTFContext). Or we could reuse 'Source' by *constructing* similar-to-dwarf source from BTF as you mentioned above, but it is somehow a waste based on you explanation in the above.
There are different ways to solve this. I would like to hear @MaskRay and @jhenderson 's opinion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D149058/new/
https://reviews.llvm.org/D149058
More information about the llvm-commits
mailing list