exec: Remove TARGET_LONG_SIZE definition

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

Philippe Mathieu-Daudé posted 5 patches 10 months, 2 weeks ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250325130221.76116-1-philmd@linaro.org

Maintainers: Richard Henderson <richard.henderson@linaro.org>, Paolo Bonzini <pbonzini@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Riku Voipio <riku.voipio@iki.fi>, Zhao Liu <zhao1.liu@intel.com>, Eduardo Habkost <eduardo@habkost.net>, Nicholas Piggin <npiggin@gmail.com>, Daniel Henrique Barboza <danielhb413@gmail.com>

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(-)

Expand all Fold all

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

Posted by Philippe Mathieu-Daudé 10 months, 2 weeks 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 10 months, 2 weeks 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 10 months, 2 weeks 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~