On 3/13/25 11:17, Richard Henderson wrote:
On 3/13/25 10:33, Pierrick Bouvier wrote:
On 3/12/25 20:44, Richard Henderson wrote:
Add a new family of translator load functions which take
an absolute endianness value in the form of MO_BE/MO_LE.
Expand the other translator_ld* functions on top of this.
Remove exec/tswap.h from translator.c.


Is there a need further down the road to break the dependency to tswap?
I am not sure of the benefit to drop tswap, as the resulting code is more 
complicated than
simply calling tswap*().

This combines the tswap in the core routine with the bswap in 
translator_ld_swap().

It enables cleanup in the various translators where we currently choose "swap 
from
TARGET_BIG_ENDIAN" rather than specifying the absolute endianness desired, 
which is
usually already at hand for use by all of the other memory references.


Ok.
Reviewed-by: Pierrick Bouvier <pierrick.bouv...@linaro.org>


r~


Reply via email to