From f73819e06379e3d015866e47e57c0cda4c34e220 Mon Sep 17 00:00:00 2001 From: Florian Krohm Date: Tue, 16 Sep 2014 21:49:45 +0000 Subject: [PATCH] In s390_decode_and_irgen don't divert the default case to a decoding error. That's wrong as the default case is never supposed to occur. In disInstr_S390_WRK use vpanic for the default case for sake of consistency. git-svn-id: svn://svn.valgrind.org/vex/trunk@2952 --- VEX/priv/guest_s390_toIR.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/VEX/priv/guest_s390_toIR.c b/VEX/priv/guest_s390_toIR.c index 2bcbf34873..65cd0e9811 100644 --- a/VEX/priv/guest_s390_toIR.c +++ b/VEX/priv/guest_s390_toIR.c @@ -16464,10 +16464,12 @@ s390_decode_and_irgen(UChar *bytes, UInt insn_length, DisResult *dres) vex_printf("unimplemented special insn: "); break; - default: case S390_DECODE_ERROR: vex_printf("decoding error: "); break; + + default: + vpanic("s390_decode_and_irgen"); } vex_printf("%02x%02x", bytes[0], bytes[1]); @@ -16550,7 +16552,7 @@ disInstr_S390_WRK(UChar *insn) } break; default: - vassert(0); + vpanic("disInstr_S390_WRK"); } } -- 2.47.2