files: 0.213
other: 0.133
semantic: 0.119
PID: 0.094
socket: 0.070
device: 0.065
graphic: 0.048
permissions: 0.047
debug: 0.045
network: 0.040
boot: 0.036
vnc: 0.035
performance: 0.030
KVM: 0.024
files: 0.206
semantic: 0.151
debug: 0.132
other: 0.095
PID: 0.077
boot: 0.069
device: 0.055
performance: 0.040
vnc: 0.039
KVM: 0.035
network: 0.035
socket: 0.024
graphic: 0.021
permissions: 0.021

qemu/target/mips/op_helper.c:943:5: style:inconclusive: Found duplicate branches for 'if' and 'else'

Source code is

    if (other_tc == other->current_tc) {
        tccause = other->CP0_Cause;
    } else {
        tccause = other->CP0_Cause;
    }

Thank you for reporting this!

Believe it or not, this has been a know issue to us for a while. We contacted the original contributor, and he says he can't recall what he actually meant when he wrote the segment. We left it as is until he remembers. However, this seems not likely to happen, and probably we will issue a fix soon. In any case, it is good that you reported it, many thanks!

You already reported that 6 months ago:
https://bugs.launchpad.net/qemu/+bug/1856706

From: Aleksandar Markovic <email address hidden>

Remove the segment:

      if (other_tc == other->current_tc) {
          tccause = other->CP0_Cause;
      } else {
          tccause = other->CP0_Cause;
      }

Original contributor can't remember what was his intention.

Fixes: 5a25ce9487 ("mips: Hook in more reg accesses via mttr/mftr")
Buglink: https://bugs.launchpad.net/qemu/+bug/1885718
Signed-off-by: Aleksandar Markovic <email address hidden>
Message-Id: <email address hidden>
Signed-off-by: Philippe Mathieu-Daudé <email address hidden>
---
Cc: Bug 1885718 <email address hidden>

 target/mips/cp0_helper.c | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/target/mips/cp0_helper.c b/target/mips/cp0_helper.c
index bbf12e4a97..de64add038 100644
--- a/target/mips/cp0_helper.c
+++ b/target/mips/cp0_helper.c
@@ -375,16 +375,9 @@ target_ulong helper_mftc0_entryhi(CPUMIPSState *env)
 target_ulong helper_mftc0_cause(CPUMIPSState *env)
 {
     int other_tc = env->CP0_VPEControl & (0xff << CP0VPECo_TargTC);
-    int32_t tccause;
     CPUMIPSState *other = mips_cpu_map_tc(env, &other_tc);
 
-    if (other_tc == other->current_tc) {
-        tccause = other->CP0_Cause;
-    } else {
-        tccause = other->CP0_Cause;
-    }
-
-    return tccause;
+    return other->CP0_Cause;
 }
 
 target_ulong helper_mftc0_status(CPUMIPSState *env)
-- 
2.21.3