Bug 444110 priv/guest_ppc_toIR.c: warning: duplicated 'if' condition

The compiler reported a duplicated condition in VEX/priv/guest_ppc_toIR.c
The handling of the plbz and xxpermx instructions have the same if/elseif
conditions.  The else if condition for the plbz instruction was wrong.  The
elseif statement should be checking for pType2 not pType1.  The plbz
instruction was inadvertently being handled by the else statement for
the lbz instruction.

This patch fixes the checking for the plbz and lbz instructions.

diff --git a/NEWS b/NEWS
index 614ce44a36fe31445995d3c24f24144c1e6f1c72..208d8afab72091961379a6c3550e7244ed1c341f 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -20,6 +20,7 @@ bugzilla (https://bugs.kde.org/enter_bug.cgi?product=valgrind) rather
 than mailing the developers (or mailing lists) directly -- bugs that
 are not entered into bugzilla tend to get forgotten about or ignored.
 
+444110 priv/guest_ppc_toIR.c:36198:31: warning: duplicated 'if' condition.
 
 To see details of a given bug, visit
   https://bugs.kde.org/show_bug.cgi?id=XXXXXX

diff --git a/VEX/priv/guest_ppc_toIR.c b/VEX/priv/guest_ppc_toIR.c
index 41d7bf65c0701c0c6d3bf10e25f5243e8f12db77..16181768e4e1507ae2bd441713ecbbe30d6ca6ec 100644 (file)
--- a/VEX/priv/guest_ppc_toIR.c
+++ b/VEX/priv/guest_ppc_toIR.c
@@ -36012,11 +36012,11 @@ DisResult disInstr_PPC_WRK (
             // splat instructions: xxpermx
             if (dis_vector_permute_prefix( prefix, theInstr, abiinfo ))
                goto decode_success;
-         } else if (is_prefix && ( ptype == pType1 ) ) {  // plbz:  load instruction
+         } else if (is_prefix && ( ptype == pType2 ) ) {  // plbz:  load instruction
             if ( !(allow_isa_3_1) ) goto decode_noIsa3_1;
             if (dis_int_load_prefix( prefix, theInstr ))
                goto decode_success;
-         } else {  // lbz:  load instruction
+         } else if (!is_prefix) {  // lbz:  load instruction
             if (dis_int_load_prefix( prefix, theInstr ))
                goto decode_success;
          }