[Lldb-commits] [PATCH] D136207: [lldb] Fix breakpoint setting so it always works when there is a line entry in a compile unit's line table.

[Pavel Labath via Phabricator via lldb-commits]

labath added inline comments.


================
Comment at: lldb/source/Symbol/CompileUnit.cpp:336
     } else {
       line_entry.range.GetBaseAddress().CalculateSymbolContext(&sc,
                                                                resolve_scope);
----------------
I think this would be more robustly written as something like
```
line_entry.range.GetBaseAddress().CalculateSymbolContext(&new_sc, resolve_scope);
if (new_sc.comp_unit == this)
  sc_list.Append(new_sc);
else
  sc_list.Append(sc);
```
-- basically using a new symbol context for the resolution, instead of twiddling the old one back and forth.


================
Comment at: lldb/test/API/functionalities/breakpoint/breakpoint_command/TestBreakpointCommand.py:109
 
+    @skipIf(oslist=["windows"])
+    @no_debug_info_test
----------------
I would hope that the yaml-based test can run on windows as well.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136207/new/

https://reviews.llvm.org/D136207