[v2] target/arm: Remove MO_TE to compile MVE/M helpers once

[PATCH v2 0/9] target/arm: Remove MO_TE to compile MVE/M helpers once

Posted by Philippe Mathieu-Daudé 2 weeks, 1 day ago

Missing review: #5

Replace build-time MO_TE definition by runtime mo_endian()
helper, allowing to compile m_helper.c and mve_helper.c once.

Philippe Mathieu-Daudé (9):
  target/arm: Ignore endianness when setting MTE tags
  target/arm: Explode MO_TExx -> MO_TE | MO_xx
  target/arm: Hoist MO_TE into mve_advance_vpt()
  target/arm: Hoist MO_TE into MVE DO_VSTR() macro
  target/arm: Introduce mo_endian() helper
  target/arm: Replace MO_TE -> mo_endian() for MVE helpers
  target/arm: Compile mve_helper.c once
  target/arm: Replace MO_TE -> mo_endian() for Cortex-M helpers
  target/arm: Compile m_helper.c once

 target/arm/internals.h      |   6 +++
 target/arm/tcg/helper-a64.c |   2 +-
 target/arm/tcg/m_helper.c   |   8 +--
 target/arm/tcg/mve_helper.c | 102 +++++++++++++++++++++---------------
 target/arm/tcg/meson.build  |   6 ++-
 5 files changed, 75 insertions(+), 49 deletions(-)

-- 
2.53.0

Re: [PATCH v2 0/9] target/arm: Remove MO_TE to compile MVE/M helpers once

Posted by Philippe Mathieu-Daudé 1 week, 5 days ago

On 14/4/26 02:53, Philippe Mathieu-Daudé wrote:

> Philippe Mathieu-Daudé (9):
>    target/arm: Ignore endianness when setting MTE tags
>    target/arm: Explode MO_TExx -> MO_TE | MO_xx
>    target/arm: Hoist MO_TE into mve_advance_vpt()
>    target/arm: Hoist MO_TE into MVE DO_VSTR() macro
>    target/arm: Introduce mo_endian() helper
>    target/arm: Replace MO_TE -> mo_endian() for MVE helpers
>    target/arm: Compile mve_helper.c once
>    target/arm: Replace MO_TE -> mo_endian() for Cortex-M helpers
>    target/arm: Compile m_helper.c once

Series fully reviewed.

Re: [PATCH v2 0/9] target/arm: Remove MO_TE to compile MVE/M helpers once

Posted by Philippe Mathieu-Daudé 2 days, 15 hours ago

On 17/4/26 05:58, Philippe Mathieu-Daudé wrote:
> On 14/4/26 02:53, Philippe Mathieu-Daudé wrote:
> 
>> Philippe Mathieu-Daudé (9):
>>    target/arm: Ignore endianness when setting MTE tags
>>    target/arm: Explode MO_TExx -> MO_TE | MO_xx
>>    target/arm: Hoist MO_TE into mve_advance_vpt()
>>    target/arm: Hoist MO_TE into MVE DO_VSTR() macro
>>    target/arm: Introduce mo_endian() helper
>>    target/arm: Replace MO_TE -> mo_endian() for MVE helpers
>>    target/arm: Compile mve_helper.c once
>>    target/arm: Replace MO_TE -> mo_endian() for Cortex-M helpers
>>    target/arm: Compile m_helper.c once
> 
> Series fully reviewed.

Checking as of master=aa15257174d, this series still applies cleanly.

Re: [PATCH v2 0/9] target/arm: Remove MO_TE to compile MVE/M helpers once

Posted by Peter Maydell 2 days, 14 hours ago

On Mon, 27 Apr 2026 at 09:15, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> On 17/4/26 05:58, Philippe Mathieu-Daudé wrote:
> > On 14/4/26 02:53, Philippe Mathieu-Daudé wrote:
> >
> >> Philippe Mathieu-Daudé (9):
> >>    target/arm: Ignore endianness when setting MTE tags
> >>    target/arm: Explode MO_TExx -> MO_TE | MO_xx
> >>    target/arm: Hoist MO_TE into mve_advance_vpt()
> >>    target/arm: Hoist MO_TE into MVE DO_VSTR() macro
> >>    target/arm: Introduce mo_endian() helper
> >>    target/arm: Replace MO_TE -> mo_endian() for MVE helpers
> >>    target/arm: Compile mve_helper.c once
> >>    target/arm: Replace MO_TE -> mo_endian() for Cortex-M helpers
> >>    target/arm: Compile m_helper.c once
> >
> > Series fully reviewed.
>
> Checking as of master=aa15257174d, this series still applies cleanly.



Applied to target-arm.next, thanks.

-- PMM