[Lldb-commits] [lldb] [lldb-dap] Add valueLocationReference for member function pointers (PR #186837)

[via lldb-commits]

================

----------------
jimingham wrote:

This seems a little incoherent to me.  

If the address isn't a load address, you return the fail value.  Then you check whether there is a process and if not return the return value you got without calling FixDataAddress.

But if there's no process, there's no way to get a load address.  Load addresses are addresses in running processes.  So I can't see how you would ever get to line 814.

There may be some intention I'm missing here, but it is not clear what that is from this code.

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