[Lldb-commits] [lldb] [lldb/Symbol] Relax LineEntry validity for PC-less frames (PR #158811)

[Greg Clayton via lldb-commits]

clayborg wrote:

It might be worth having an extra bool in `LineEntry` that states if the line entry doesn't have an address range? Or mark the `LineEntry` as a synthetic line entry to indicate it is ok for it to not have an address range. Then any code that needs to deal with such a line entry would use different accessors to avoid changing the nature of the current `LineEntry::IsValid()`. Any clients that want to get around this, can then call a new function like `LineEntry::IsValidSynthetic()` which could ignore the line entry's address range, but only if a bool in the line entry marks the line entry as not needing an address range. 

https://github.com/llvm/llvm-project/pull/158811