[Lldb-commits] [lldb] [lldb] Make LineEntry AddressRange member optional (PR #158811)

Med Ismail Bennani via lldb-commits lldb-commits at lists.llvm.org
Tue Dec 2 12:58:58 PST 2025


medismailben wrote:

> > Can we break this up into an NFC patch that makes `LineEntry` a class and a separate PR for the functional change that makes `AddressRange` optional?
> 
> +1.
> 
> Why do you want to turn it into a class? In C++, the only difference between a class and a struct is the default access level. Structs are public by default, Classes are private by default.

To Alex's point, this didn't require to turn the `LineEntry` struct in to a class so I just made the `range` member a private optional and added public getter/setter methods.

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


More information about the lldb-commits mailing list