[Lldb-commits] [PATCH] D20835: Mark the current column when displaying the context of the current breakpoint on the terminal.
Todd Fiala via lldb-commits
lldb-commits at lists.llvm.org
Tue Sep 20 21:59:32 PDT 2016
tfiala marked an inline comment as done.
================
Comment at: source/Core/Disassembler.cpp:292-294
@@ -291,2 +291,5 @@
decl_line.line = func_decl_line;
+ // TODO do we care about column on these entries? If so, we need to
+ // plumb that through GetStartLineSourceInfo.
+ decl_line.column = 0;
}
----------------
I will leave this for a potential follow-up patch at another time. At that point, I would suggest enabling stop line and column marking on the "source list" command, which currently doesn't mark up a displayed line that coincides with any currently-stopped thread.
I'll strip out this TODO on the final check-in.
https://reviews.llvm.org/D20835
More information about the lldb-commits
mailing list