[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