[Lldb-commits] [lldb] [lldb] Refactor away UB in SBValue::GetLoadAddress (PR #141799)

[via lldb-commits]

jimingham wrote:

It always makes me a little sad to see `.first` or `.second` showing up - they are so non-self-documenting.  How horrible would this be if you used a `struct AddrAndType { AddressType type; lldb::addr_t address}` (I'm a bit surprised we don't have one already) instead so that you would do `GetLoadAddress().address` rather than `GetLoadAddress().second`?

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