diff mbox series

[v5,09/23] target/ppc: Introduce macros to check isa extensions

Message ID 20210517205025.3777947-10-matheus.ferst@eldorado.org.br
State Superseded
Headers show
Series [v5,01/23] target/ppc: Introduce gen_icount_io_start | expand

Commit Message

Matheus K. Ferst May 17, 2021, 8:50 p.m. UTC
From: Richard Henderson <richard.henderson@linaro.org>


These will be used by the decodetree trans_* functions
to early-exit when the instruction set is not enabled.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

Signed-off-by: Matheus Ferst <matheus.ferst@eldorado.org.br>

---
 target/ppc/translate.c | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

-- 
2.25.1

Comments

David Gibson May 18, 2021, 12:21 a.m. UTC | #1
On Mon, May 17, 2021 at 05:50:11PM -0300, matheus.ferst@eldorado.org.br wrote:
> From: Richard Henderson <richard.henderson@linaro.org>

> 

> These will be used by the decodetree trans_* functions

> to early-exit when the instruction set is not enabled.

> 

> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

> Signed-off-by: Matheus Ferst <matheus.ferst@eldorado.org.br>


Applied to ppc-for-6.1, thanks.

> ---

>  target/ppc/translate.c | 26 ++++++++++++++++++++++++++

>  1 file changed, 26 insertions(+)

> 

> diff --git a/target/ppc/translate.c b/target/ppc/translate.c

> index ea200f9637..dc0f5fafc2 100644

> --- a/target/ppc/translate.c

> +++ b/target/ppc/translate.c

> @@ -7750,6 +7750,32 @@ static inline void set_avr64(int regno, TCGv_i64 src, bool high)

>      tcg_gen_st_i64(src, cpu_env, avr64_offset(regno, high));

>  }

>  

> +/*

> + * Helpers for trans_* functions to check for specific insns flags.

> + * Use token pasting to ensure that we use the proper flag with the

> + * proper variable.

> + */

> +#define REQUIRE_INSNS_FLAGS(CTX, NAME) \

> +    do {                                                \

> +        if (((CTX)->insns_flags & PPC_##NAME) == 0) {   \

> +            return false;                               \

> +        }                                               \

> +    } while (0)

> +

> +#define REQUIRE_INSNS_FLAGS2(CTX, NAME) \

> +    do {                                                \

> +        if (((CTX)->insns_flags2 & PPC2_##NAME) == 0) { \

> +            return false;                               \

> +        }                                               \

> +    } while (0)

> +

> +/* Then special-case the check for 64-bit so that we elide code for ppc32. */

> +#if TARGET_LONG_BITS == 32

> +# define REQUIRE_64BIT(CTX)  return false

> +#else

> +# define REQUIRE_64BIT(CTX)  REQUIRE_INSNS_FLAGS(CTX, 64B)

> +#endif

> +

>  #include "translate/fp-impl.c.inc"

>  

>  #include "translate/vmx-impl.c.inc"


-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson
diff mbox series

Patch

diff --git a/target/ppc/translate.c b/target/ppc/translate.c
index ea200f9637..dc0f5fafc2 100644
--- a/target/ppc/translate.c
+++ b/target/ppc/translate.c
@@ -7750,6 +7750,32 @@  static inline void set_avr64(int regno, TCGv_i64 src, bool high)
     tcg_gen_st_i64(src, cpu_env, avr64_offset(regno, high));
 }
 
+/*
+ * Helpers for trans_* functions to check for specific insns flags.
+ * Use token pasting to ensure that we use the proper flag with the
+ * proper variable.
+ */
+#define REQUIRE_INSNS_FLAGS(CTX, NAME) \
+    do {                                                \
+        if (((CTX)->insns_flags & PPC_##NAME) == 0) {   \
+            return false;                               \
+        }                                               \
+    } while (0)
+
+#define REQUIRE_INSNS_FLAGS2(CTX, NAME) \
+    do {                                                \
+        if (((CTX)->insns_flags2 & PPC2_##NAME) == 0) { \
+            return false;                               \
+        }                                               \
+    } while (0)
+
+/* Then special-case the check for 64-bit so that we elide code for ppc32. */
+#if TARGET_LONG_BITS == 32
+# define REQUIRE_64BIT(CTX)  return false
+#else
+# define REQUIRE_64BIT(CTX)  REQUIRE_INSNS_FLAGS(CTX, 64B)
+#endif
+
 #include "translate/fp-impl.c.inc"
 
 #include "translate/vmx-impl.c.inc"