[Valgrind-developers] [valgrind] Bug 503677 - duplicated-cond compiler warning in dis_RV64M

[Paul F. <pa...@so...>]

https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=09c161e6e0ee8ee057d81cfb71c0d98b16f36a19

commit 09c161e6e0ee8ee057d81cfb71c0d98b16f36a19
Author: Paul Floyd <pj...@wa...>
Date:   Sun May 4 08:55:45 2025 +0200

    Bug 503677 - duplicated-cond compiler warning in dis_RV64M

Diff:
---
 NEWS                          | 1 +
 VEX/priv/guest_riscv64_toIR.c | 2 --
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/NEWS b/NEWS
index 71c389c535..b69dc84f8b 100644
--- a/NEWS
+++ b/NEWS
@@ -24,6 +24,7 @@ than mailing the developers (or mailing lists) directly -- bugs that
 are not entered into bugzilla tend to get forgotten about or ignored.
 
 503641  close_range syscalls started failing with 3.25.0
+503677  duplicated-cond compiler warning in dis_RV64M
 
 To see details of a given bug, visit
   https://bugs.kde.org/show_bug.cgi?id=XXXXXX
diff --git a/VEX/priv/guest_riscv64_toIR.c b/VEX/priv/guest_riscv64_toIR.c
index 5d9b903c9b..393a7ca7d0 100644
--- a/VEX/priv/guest_riscv64_toIR.c
+++ b/VEX/priv/guest_riscv64_toIR.c
@@ -1754,8 +1754,6 @@ static Bool dis_RV64M(/*MB_OUT*/ DisResult* dres,
       UInt rs1    = INSN(19, 15);
       UInt rs2    = INSN(24, 20);
       if (funct3 == 0b010) {
-         /* Invalid {MUL,DIV,REM}<x>, fall through. */
-      } else if (funct3 == 0b010) {
          /* MULHSU, not currently handled, fall through. */
       } else {
          if (rd != 0) {