diff mbox series

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

Message ID 20210604183506.916654-3-richard.henderson@linaro.org
State Superseded
Headers show
Series target/arm: Decode fixes for aarch64 | expand

Commit Message

Richard Henderson June 4, 2021, 6:35 p.m. UTC 
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(-)

-- 
2.25.1

Comments

Philippe Mathieu-Daudé June 6, 2021, 6:46 p.m. UTC | #1 
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>
diff mbox series

Patch

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();
     }