Re: [PATCH v2 2/3] target/arm/tcg/m_helper.c: Include the full helpers only with CONFIG_ARM_V7M

2024-03-08 Thread Thomas Huth
On 08/03/2024 15.00, Peter Maydell wrote: On Fri, 8 Mar 2024 at 12:54, Thomas Huth wrote: I gave it a try, but then we end up again with the problem that I already mentioned in the discussion about patch 1: CONFIG_ARM_V7M is not set for the linux-user binaries, so m_helper.c would not get inclu

Re: [PATCH v2 2/3] target/arm/tcg/m_helper.c: Include the full helpers only with CONFIG_ARM_V7M

2024-03-08 Thread Peter Maydell
On Fri, 8 Mar 2024 at 12:54, Thomas Huth wrote: > I gave it a try, but then we end up again with the problem that I already > mentioned in the discussion about patch 1: CONFIG_ARM_V7M is not set for the > linux-user binaries, so m_helper.c would not get included there anymore and > we end up with

Re: [PATCH v2 2/3] target/arm/tcg/m_helper.c: Include the full helpers only with CONFIG_ARM_V7M

2024-03-08 Thread Thomas Huth
On 04/03/2024 16.22, Peter Maydell wrote: On Thu, 1 Feb 2024 at 19:12, Thomas Huth wrote: On 01/02/2024 15.19, Peter Maydell wrote: On Mon, 29 Jan 2024 at 08:18, Thomas Huth wrote: If CONFIG_ARM_V7M is not set, we don't want to include the full-fledged helper functions that require additio

Re: [PATCH v2 2/3] target/arm/tcg/m_helper.c: Include the full helpers only with CONFIG_ARM_V7M

2024-03-04 Thread Peter Maydell
On Thu, 1 Feb 2024 at 19:12, Thomas Huth wrote: > > On 01/02/2024 15.19, Peter Maydell wrote: > > On Mon, 29 Jan 2024 at 08:18, Thomas Huth wrote: > >> > >> If CONFIG_ARM_V7M is not set, we don't want to include the full-fledged > >> helper functions that require additional functions for linking.

Re: [PATCH v2 2/3] target/arm/tcg/m_helper.c: Include the full helpers only with CONFIG_ARM_V7M

2024-02-01 Thread Thomas Huth
On 01/02/2024 15.19, Peter Maydell wrote: On Mon, 29 Jan 2024 at 08:18, Thomas Huth wrote: If CONFIG_ARM_V7M is not set, we don't want to include the full-fledged helper functions that require additional functions for linking. The reduced set of the linux-user functions works fine as stubs in

Re: [PATCH v2 2/3] target/arm/tcg/m_helper.c: Include the full helpers only with CONFIG_ARM_V7M

2024-02-01 Thread Peter Maydell
On Mon, 29 Jan 2024 at 08:18, Thomas Huth wrote: > > If CONFIG_ARM_V7M is not set, we don't want to include the full-fledged > helper functions that require additional functions for linking. The > reduced set of the linux-user functions works fine as stubs in this > case, so change the #ifdef stat

[PATCH v2 2/3] target/arm/tcg/m_helper.c: Include the full helpers only with CONFIG_ARM_V7M

2024-01-29 Thread Thomas Huth
If CONFIG_ARM_V7M is not set, we don't want to include the full-fledged helper functions that require additional functions for linking. The reduced set of the linux-user functions works fine as stubs in this case, so change the #ifdef statement accordingly. Signed-off-by: Thomas Huth --- target/