jasonmolenda wrote:

I wanted to add a new `eBinaryType` enum from the ObjectFile to describe what 
address is being returned.  But within ProcessMachCore, there was no need to 
add the distinction between the existing `m_dyld_addr` and the new 
`m_dyld_all_image_infos_addr`, I did it to keep the meaning of the address 
clear at this layer, and to make logging more clear.  But ProcessMachCore 
returns either address to DynamicLoaderMacOSX, and it sniffs for a Mach-O magic 
number in the first 4 bytes to disambiguate between them already.  I could have 
ProcessMachCore put either value in `m_dyld_addr` and everything would still 
work. 

https://github.com/llvm/llvm-project/pull/127156
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to