wallace marked an inline comment as done. wallace added inline comments.
================ Comment at: lldb/tools/lldb-vscode/JSONUtils.cpp:332 + + object.try_emplace("line", line); + ---------------- clayborg wrote: > This should do what it was doing before: grab the source line only from the > address. Why? If you set a breakpoint on a line that has no code like: > > ``` > 1 int main(int argc, ...) > 2 { /// <-- set bp here > 3 int foo = arg * 10; /// <-- bp will end up here > ``` > > So we should still be looking up the line number from the Address of the > first resolved location so that the source line does get adjusted in the IDE. TIL Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D76968/new/ https://reviews.llvm.org/D76968 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits