[Lldb-commits] [PATCH] D98619: [lldb] Use CompileUnit::ResolveSymbolContext in SymbolFileDWARF
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Mar 22 00:50:36 PDT 2021
labath added a comment.
In D98619#2634472 <https://reviews.llvm.org/D98619#2634472>, @kimanh wrote:
> Thanks a lot for the review Pavel! I've updated the test. If it looks fine like this, could you help me to commit this change?
Done. Thanks for the patch.
================
Comment at: lldb/test/Shell/SymbolFile/DWARF/dwarf5-debug_line-file-index.s:98
+ .byte 86
+ .byte 3 # DW_AT_name
+ .byte 0 # DW_AT_decl_file
----------------
kimanh wrote:
> labath wrote:
> > I was puzzled by this line as the string 3 is nowhere to be found. I take it this was manually reduced? If so, I think you should be able to delete the entire subprogram and base_type tags (and the relevant abbreviations), as that is not necessary for the line tables to work.
> Thanks for the guidance here! Yes, that's correct, I've manually reduced it but wasn't sure what else I could safely remove. I've now removed the subprogram parts, and the base_type tags. Hope that looks better now!
This looks better. Thanks. FTR, I don't think it's necessary to produce absolutely minimal inputs (though the easy-to-do reductions are definitely nice to have). I just don't want to have invalid inputs (like these missing cross-references) in the test file (unless of course, that is what you're testing, but that's not the case here).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98619/new/
https://reviews.llvm.org/D98619
More information about the lldb-commits
mailing list