================
@@ -508,8 +508,20 @@ void BreakpointLocation::GetDescription(Stream *s,
s->PutCString("re-exported target = ");
else
s->PutCString("where = ");
+
+ // If there's a preferred line entry for printing, use that.
+ bool show_function_info = true;
+ if (auto preferred = GetPreferredLineEntry()) {
+ sc.line_entry = *preferred;
+ // FIXME: We're going to get the function name wrong when the preferred
+ // line entry is not the lowest one. For now, just leave the function
+ // out in this case, but we really should also figure out how to easily
+ // fake the function name here.
----------------
jimingham wrote:
What would it mean to replace the Module, the CompileUnit or the Function?
This is for a breakpoint location, so whatever we do has to result in this
being the same address. I was hesitant to introduce a more general "breakpoint
gets to modify its symbol context" if I didn't know what it meant or how to
ensure we didn't end up with a SymbolContext that wasn't at the breakpoint
location's address. I was originally planning to JUST store the "inlined call
stack depth" to make it clear this couldn't change the PC value, but that ended
up being more annoying that helpful.
https://github.com/llvm/llvm-project/pull/112939
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
