[Lldb-commits] [lldb] [lldb-dap] Add 'source' references to stack frames without source files. (PR #128268)
Jonas Devlieghere via lldb-commits
lldb-commits at lists.llvm.org
Sat Feb 22 12:25:17 PST 2025
================
@@ -698,14 +697,22 @@ llvm::json::Value CreateSource(llvm::StringRef source_path) {
return llvm::json::Value(std::move(source));
}
-static std::optional<llvm::json::Value> CreateSource(lldb::SBFrame &frame) {
+static llvm::json::Value CreateSource(lldb::SBFrame &frame,
----------------
JDevlieghere wrote:
I assume this function is used elsewhere? The line number and column have a different meaning based on whether we have source info and having the logic spread across this function and `CreateStackFrame` seems like a pretty big foot gun. Any chance we can localize this logic?
https://github.com/llvm/llvm-project/pull/128268
More information about the lldb-commits
mailing list