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

Philippe Mathieu-Daudé posted 5 patches 6 days, 18 hours ago
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(-)
[PATCH-for-10.1 0/5] exec: Remove TARGET_LONG_SIZE definition
Posted by Philippe Mathieu-Daudé 6 days, 18 hours ago
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(-)

-- 
2.47.1


Re: [PATCH-for-10.1 0/5] exec: Remove TARGET_LONG_SIZE definition
Posted by Pierrick Bouvier 6 days, 17 hours ago
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>
Re: [PATCH-for-10.1 0/5] exec: Remove TARGET_LONG_SIZE definition
Posted by Richard Henderson 6 days, 17 hours ago
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~