https://github.com/DavidSpickett commented:

The overall logic makes sense to me but others have the expertise to approve.

Arm 32-bit is usually sensitive to these changes because it needs to know what 
code regions are Thumb vs. Arm code. IIRC this uses a combination of mapping 
symbols and function symbol addresses.

I haven't looked too closely at this change but it seems likely that it would 
be at worst neutral and quite likely an improvement for Arm.

Our bot is good at catching this stuff so if it fails, I'll revert this and 
figure out the problem for you.

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

Reply via email to