mbox series

[PATCH-for-10.1,0/5] exec: Remove TARGET_LONG_SIZE definition

Message ID 20250325130221.76116-1-philmd@linaro.org
Headers show
Series exec: Remove TARGET_LONG_SIZE definition | expand

Message

Philippe Mathieu-Daudé March 25, 2025, 1:02 p.m. UTC 
TARGET_LONG_SIZE is only used in 1 source file
and 1 header. Remove it to be able to focus on
making TARGET_LONG_BITS target agnostic.

Philippe Mathieu-Daudé (5):
  target/i386: Use explicit little-endian LD/ST API
  gdbstub: Remove ldtul_be_p() and ldtul_le_p() macros
  target/ppc/gdbstub: Replace TARGET_LONG_SIZE -> sizeof(target_ulong)
  user/tswap: Replace TARGET_LONG_SIZE -> TARGET_LONG_BITS
  exec/target_long: Remove TARGET_LONG_SIZE definition

 include/exec/target_long.h           |  8 +++----
 include/gdbstub/helpers.h            |  4 ----
 include/user/tswap-target.h          |  2 +-
 target/i386/gdbstub.c                | 24 ++++++++++-----------
 target/i386/tcg/system/excp_helper.c |  4 ++--
 target/i386/xsave_helper.c           | 32 ++++++++++++++--------------
 target/ppc/gdbstub.c                 | 10 ++++-----
 7 files changed, 39 insertions(+), 45 deletions(-)

Comments

Richard Henderson March 25, 2025, 2:15 p.m. UTC | #1 
On 3/25/25 06:02, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (5):
>    target/i386: Use explicit little-endian LD/ST API
>    gdbstub: Remove ldtul_be_p() and ldtul_le_p() macros
>    target/ppc/gdbstub: Replace TARGET_LONG_SIZE -> sizeof(target_ulong)
>    user/tswap: Replace TARGET_LONG_SIZE -> TARGET_LONG_BITS
>    exec/target_long: Remove TARGET_LONG_SIZE definition

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~
Pierrick Bouvier March 25, 2025, 2:25 p.m. UTC | #2 
On 3/25/25 06:02, Philippe Mathieu-Daudé wrote:
> TARGET_LONG_SIZE is only used in 1 source file
> and 1 header. Remove it to be able to focus on
> making TARGET_LONG_BITS target agnostic.
> 
> Philippe Mathieu-Daudé (5):
>    target/i386: Use explicit little-endian LD/ST API
>    gdbstub: Remove ldtul_be_p() and ldtul_le_p() macros
>    target/ppc/gdbstub: Replace TARGET_LONG_SIZE -> sizeof(target_ulong)
>    user/tswap: Replace TARGET_LONG_SIZE -> TARGET_LONG_BITS
>    exec/target_long: Remove TARGET_LONG_SIZE definition
> 

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