aprantl added inline comments.
================ Comment at: lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.cpp:1029 + return false; + + case DW_TAG_compile_unit: ---------------- davide wrote: > aprantl wrote: > > To save everyone the trouble: > > ``` > > 0x0000002a: DW_TAG_subprogram > > DW_AT_low_pc (0x0000000000000000) > > DW_AT_high_pc (0x0000000000000006) > > DW_AT_frame_base (DW_OP_reg6 RBP) > > DW_AT_name ("f") > > DW_AT_decl_file ("/tmp/t.c") > > DW_AT_decl_line (1) > > DW_AT_external (true) > > > > 0x0000003f: DW_TAG_variable > > DW_AT_name ("g_i") > > DW_AT_type (0x00000055 "int") > > DW_AT_decl_file ("/tmp/t.c") > > DW_AT_decl_line (2) > > DW_AT_location (DW_OP_addr 0x280) > > > > 0x00000054: NULL > > ``` > hmm, so, what's the DWARF'y way to find out whether a variable is static? > This code is just moved from another place which did this. Looks like the function's name is just confusing. We should add a doxgygen comment for it, CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77698/new/ https://reviews.llvm.org/D77698 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits