[lldb/Symbol] Relax LineEntry validity for PC-less frames

LineEntry::IsValid() no longer requires a valid address range, enabling
line entries created using the SBAPI to be set in SBSymbolContext.

Signed-off-by: Med Ismail Bennani <[email protected]>

Author: medismailben

lldb/source/API/SBLineEntry.cpp

@@ -50,7 +50,7 @@ SBAddress SBLineEntry::GetStartAddress() const {
   LLDB_INSTRUMENT_VA(this);
 
   SBAddress sb_address;
-  if (m_opaque_up)
+  if (m_opaque_up && m_opaque_up->range.IsValid())
     sb_address.SetAddress(m_opaque_up->range.GetBaseAddress());
 
   return sb_address;
@@ -60,7 +60,7 @@ SBAddress SBLineEntry::GetEndAddress() const {
   LLDB_INSTRUMENT_VA(this);
 
   SBAddress sb_address;
-  if (m_opaque_up) {
+  if (m_opaque_up && m_opaque_up->range.IsValid()) {
     sb_address.SetAddress(m_opaque_up->range.GetBaseAddress());
     sb_address.OffsetAddress(m_opaque_up->range.GetByteSize());
   }

lldb/source/Symbol/LineEntry.cpp

@@ -32,9 +32,7 @@ void LineEntry::Clear() {
   is_terminal_entry = 0;
 }
 
-bool LineEntry::IsValid() const {
-  return range.GetBaseAddress().IsValid() && line != LLDB_INVALID_LINE_NUMBER;
-}
+bool LineEntry::IsValid() const { return line != LLDB_INVALID_LINE_NUMBER; }
 
 bool LineEntry::DumpStopContext(Stream *s, bool show_fullpaths) const {
   const FileSpec &file = file_sp->GetSpecOnly();

lldb/test/API/functionalities/scripted_process/dummy_scripted_process.py

@@ -74,6 +74,20 @@ def __init__(self, process, args):
         self.frames.append(DummyScriptedFrame(self, args, len(self.frames), "bar"))
         self.frames.append(DummyScriptedFrame(self, args, len(self.frames), "foo"))
 
+        cwd = os.path.dirname(os.path.abspath(__file__))
+
+        le = lldb.SBLineEntry()
+        le.SetFileSpec(lldb.SBFileSpec(os.path.join(cwd, "baz.cpp"), True))
+        le.SetLine(9)
+        le.SetColumn(10)
+
+        sym_ctx = lldb.SBSymbolContext()
+        sym_ctx.SetLineEntry(le)
+
+        self.frames.append(
+            DummyScriptedFrame(self, args, len(self.frames), "baz", sym_ctx)
+        )
+
     def get_thread_id(self) -> int:
         return 0x19