device: 0.684
socket: 0.683
ppc: 0.681
kernel: 0.668
PID: 0.668
network: 0.661
register: 0.629
performance: 0.612
hypervisor: 0.610
architecture: 0.607
peripherals: 0.605
VMM: 0.568
permissions: 0.566
user-level: 0.540
arm: 0.539
vnc: 0.538
boot: 0.522
i386: 0.517
semantic: 0.517
TCG: 0.513
x86: 0.497
mistranslation: 0.491
assembly: 0.460
risc-v: 0.458
graphic: 0.457
KVM: 0.447
files: 0.446
debug: 0.423
virtual: 0.363

target/mips/op_helper.c:971:duplicated branches ?

qemu-4.2.0/target/mips/op_helper.c:971:8: warning: this condition has identical branches [-Wduplicated-branches]

Source code is

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

Possible cut'n'paste error ?

On Tue, 17 Dec. 2019, 09:28 Thomas Huth, <email address hidden> wrote:

> On 17/12/2019 14.25, dcb wrote:
> > Public bug reported:
> >
> > qemu-4.2.0/target/mips/op_helper.c:971:8: warning: this condition has
> > identical branches [-Wduplicated-branches]
> >
> > Source code is
> >
> >    if (other_tc == other->current_tc) {
> >         tccause = other->CP0_Cause;
> >     } else {
> >         tccause = other->CP0_Cause;
> >     }
> >
> > Possible cut'n'paste error ?
>
>  Hi Edgar,
>
> you added the code in 2011 ... do you still remember the details here,
> whether that was on purpose or whether it should look differently?
>
>  Thomas
>


Hi Thomas,

Sorry but I barely recall yesterday, 2011 was a long time ago... It does
look weird.

IIRC smp was kinda of working but the multithreading support wasn't. Some
of these paths were probably not tested too much at the time.

Cheers,
Edgar

>
>


Patch has been included here:
https://git.qemu.org/?p=qemu.git;a=commitdiff;h=9788e8c9b64e4cebb01