[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] target-arm/translate.c: Use arm_dc_feature(
From: |
Claudio Fontana |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] target-arm/translate.c: Use arm_dc_feature() in ENABLE_ARCH_ macros |
Date: |
Tue, 4 Nov 2014 10:04:20 +0100 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; rv:24.0) Gecko/20100101 Thunderbird/24.0.1 |
On 28.10.2014 20:24, Peter Maydell wrote:
> All the places where we use the ENABLE_ARCH_* and ARCH() macros have a
> DisasContext* s, so switch them over to use arm_dc_feature() rather than
> arm_feature() so we don't need to pass the CPUARMState* env around too.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> target-arm/translate.c | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/target-arm/translate.c b/target-arm/translate.c
> index 1d52e47..f69e5ef 100644
> --- a/target-arm/translate.c
> +++ b/target-arm/translate.c
> @@ -38,16 +38,16 @@
> #include "trace-tcg.h"
>
>
> -#define ENABLE_ARCH_4T arm_feature(env, ARM_FEATURE_V4T)
> -#define ENABLE_ARCH_5 arm_feature(env, ARM_FEATURE_V5)
> +#define ENABLE_ARCH_4T arm_dc_feature(s, ARM_FEATURE_V4T)
> +#define ENABLE_ARCH_5 arm_dc_feature(s, ARM_FEATURE_V5)
> /* currently all emulated v5 cores are also v5TE, so don't bother */
> -#define ENABLE_ARCH_5TE arm_feature(env, ARM_FEATURE_V5)
> +#define ENABLE_ARCH_5TE arm_dc_feature(s, ARM_FEATURE_V5)
> #define ENABLE_ARCH_5J 0
> -#define ENABLE_ARCH_6 arm_feature(env, ARM_FEATURE_V6)
> -#define ENABLE_ARCH_6K arm_feature(env, ARM_FEATURE_V6K)
> -#define ENABLE_ARCH_6T2 arm_feature(env, ARM_FEATURE_THUMB2)
> -#define ENABLE_ARCH_7 arm_feature(env, ARM_FEATURE_V7)
> -#define ENABLE_ARCH_8 arm_feature(env, ARM_FEATURE_V8)
> +#define ENABLE_ARCH_6 arm_dc_feature(s, ARM_FEATURE_V6)
> +#define ENABLE_ARCH_6K arm_dc_feature(s, ARM_FEATURE_V6K)
> +#define ENABLE_ARCH_6T2 arm_dc_feature(s, ARM_FEATURE_THUMB2)
> +#define ENABLE_ARCH_7 arm_dc_feature(s, ARM_FEATURE_V7)
> +#define ENABLE_ARCH_8 arm_dc_feature(s, ARM_FEATURE_V8)
>
> #define ARCH(x) do { if (!ENABLE_ARCH_##x) goto illegal_op; } while(0)
>
>
nit: the line for ENABLE_ARCH_6K seems to be differently aligned than the rest.
Reviewed-by: Claudio Fontana <address@hidden>
--
Claudio Fontana
Server Virtualization Architect
Huawei Technologies Duesseldorf GmbH
Riesstraße 25 - 80992 München
office: +49 89 158834 4135
mobile: +49 15253060158
- Re: [Qemu-devel] [PATCH 1/5] target-arm/translate.c: Use arm_dc_feature() in ENABLE_ARCH_ macros,
Claudio Fontana <=