DavidSpickett wrote:

I've just merged the test case for the existing behaviour.

If you still want to pursue this PR please see if Jason's suggestion could work:
> The patch, currently is basically "If Symbol::GetLoadAddress failed, get the 
> file address and use it as-is, we'll just assume this is an absolute VA". I 
> think this should be handled correctly in Symbol::GetLoadAddress, or else 
> we'll have other callers that will misbehave with an absolute VA in a Symbol, 
> like this.

I don't have the time at the moment to understand it myself, but if you have 
trouble with it I might be able to find some time eventually.

https://github.com/llvm/llvm-project/pull/200134
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to