diff mbox series

[01/10] target/arm: Remove unused GEN_NEON_INTEGER_OP macro

Message ID 20200515142056.21346-2-peter.maydell@linaro.org
State Superseded
Headers show
Series target/arm: Convert 2-reg-shift and 1-reg-imm Neon insns to decodetree | expand

Commit Message

Peter Maydell May 15, 2020, 2:20 p.m. UTC
The GEN_NEON_INTEGER_OP macro is no longer used; remove it.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

---
Between Richard's cleanup and mine we deleted all the uses of this,
but since neither series on its own was sufficient to delete all
of them we failed to remove the macro definition when it finally
became unused.
---
 target/arm/translate.c | 23 -----------------------
 1 file changed, 23 deletions(-)

-- 
2.20.1

Comments

Richard Henderson May 15, 2020, 10:07 p.m. UTC | #1
On 5/15/20 7:20 AM, Peter Maydell wrote:
> The GEN_NEON_INTEGER_OP macro is no longer used; remove it.

> 

> Signed-off-by: Peter Maydell <peter.maydell@linaro.org>

> ---

> Between Richard's cleanup and mine we deleted all the uses of this,

> but since neither series on its own was sufficient to delete all

> of them we failed to remove the macro definition when it finally

> became unused.

> ---

>  target/arm/translate.c | 23 -----------------------

>  1 file changed, 23 deletions(-)


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



r~
diff mbox series

Patch

diff --git a/target/arm/translate.c b/target/arm/translate.c
index 4c9bb8b5ac0..c8296116d4b 100644
--- a/target/arm/translate.c
+++ b/target/arm/translate.c
@@ -3034,29 +3034,6 @@  static inline void gen_neon_rsb(int size, TCGv_i32 t0, TCGv_i32 t1)
     default: return 1; \
     }} while (0)
 
-#define GEN_NEON_INTEGER_OP(name) do { \
-    switch ((size << 1) | u) { \
-    case 0: \
-        gen_helper_neon_##name##_s8(tmp, tmp, tmp2); \
-        break; \
-    case 1: \
-        gen_helper_neon_##name##_u8(tmp, tmp, tmp2); \
-        break; \
-    case 2: \
-        gen_helper_neon_##name##_s16(tmp, tmp, tmp2); \
-        break; \
-    case 3: \
-        gen_helper_neon_##name##_u16(tmp, tmp, tmp2); \
-        break; \
-    case 4: \
-        gen_helper_neon_##name##_s32(tmp, tmp, tmp2); \
-        break; \
-    case 5: \
-        gen_helper_neon_##name##_u32(tmp, tmp, tmp2); \
-        break; \
-    default: return 1; \
-    }} while (0)
-
 static TCGv_i32 neon_load_scratch(int scratch)
 {
     TCGv_i32 tmp = tcg_temp_new_i32();