[PATCH 1/2] target/arm: Only include tcg/oversized-guest.h if CONFIG_TCG

Richard Henderson posted 2 patches 2 years, 8 months ago
Maintainers: "Alex Bennée" <alex.bennee@linaro.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Thomas Huth <thuth@redhat.com>, Wainer dos Santos Moschetta <wainersm@redhat.com>, Beraldo Leal <bleal@redhat.com>, Peter Maydell <peter.maydell@linaro.org>
[PATCH 1/2] target/arm: Only include tcg/oversized-guest.h if CONFIG_TCG
Posted by Richard Henderson 2 years, 8 months ago
Fixes the build for --disable-tcg.

This header is only needed for cross-hosting.  Without CONFIG_TCG,
we know this is an AArch64 host, CONFIG_ATOMIC64 will be set, and
the TCG_OVERSIZED_GUEST block will never be compiled.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 target/arm/ptw.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/target/arm/ptw.c b/target/arm/ptw.c
index b2dc223525..37bcb17a9e 100644
--- a/target/arm/ptw.c
+++ b/target/arm/ptw.c
@@ -14,8 +14,9 @@
 #include "cpu.h"
 #include "internals.h"
 #include "idau.h"
-#include "tcg/oversized-guest.h"
-
+#ifdef CONFIG_TCG
+# include "tcg/oversized-guest.h"
+#endif
 
 typedef struct S1Translate {
     ARMMMUIdx in_mmu_idx;
-- 
2.34.1
Re: [PATCH 1/2] target/arm: Only include tcg/oversized-guest.h if CONFIG_TCG
Posted by Philippe Mathieu-Daudé 2 years, 8 months ago
On 7/6/23 00:46, Richard Henderson wrote:
> Fixes the build for --disable-tcg.
> 
> This header is only needed for cross-hosting.  Without CONFIG_TCG,
> we know this is an AArch64 host, CONFIG_ATOMIC64 will be set, and
> the TCG_OVERSIZED_GUEST block will never be compiled.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   target/arm/ptw.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>