[PATCH 2/3] target/arm: Remove fprintf from disas_simd_mod_imm

Richard Henderson posted 3 patches 4 years, 8 months ago
Maintainers: Peter Maydell <peter.maydell@linaro.org>
[PATCH 2/3] target/arm: Remove fprintf from disas_simd_mod_imm
Posted by Richard Henderson 4 years, 8 months ago
The default of this switch is truly unreachable.
The switch selector is 3 bits, and all 8 cases are present.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
 target/arm/translate-a64.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/target/arm/translate-a64.c b/target/arm/translate-a64.c
index 2477b55c53..9bb15ca618 100644
--- a/target/arm/translate-a64.c
+++ b/target/arm/translate-a64.c
@@ -8291,7 +8291,6 @@ static void disas_simd_mod_imm(DisasContext *s, uint32_t insn)
         }
         break;
     default:
-        fprintf(stderr, "%s: cmode_3_1: %x\n", __func__, cmode_3_1);
         g_assert_not_reached();
     }
 
-- 
2.25.1


Re: [PATCH 2/3] target/arm: Remove fprintf from disas_simd_mod_imm
Posted by Philippe Mathieu-Daudé 4 years, 8 months ago
On 6/4/21 8:35 PM, Richard Henderson wrote:
> The default of this switch is truly unreachable.
> The switch selector is 3 bits, and all 8 cases are present.
> 
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>  target/arm/translate-a64.c | 1 -
>  1 file changed, 1 deletion(-)

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>