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

Reply via email to