On 21/07/2023 09.03, Olaf Hering wrote:
Fri, 21 Jul 2023 08:54:21 +0200 Thomas Huth <th...@redhat.com>:

Which compiler version is causing trouble for you?

Right now it is gcc 13, hopefully every compiler will error out with 
-Werror=return-type.

I don't think this will happen - otherwise we would have seen this already. It rather looks like your compiler version is missing something here - have a look at the affected function:

static float64_t f64Random( void )
{

    switch ( random_ui8() & 7 ) {
     case 0:
     case 1:
     case 2:
        return f64RandomQOutP3();
     case 3:
        return f64RandomQOutPInf();
     case 4:
     case 5:
     case 6:
        return f64RandomQInfP3();
     case 7:
        return f64RandomQInfPInf();
    }

}

The argument in the switch statement is limited with "& 7" to the range of 0 ... 7 , so there is no way the control flow can really reach the bottom of the function. I wonder why your GCC 13 gets that wrong, all other versions seem to be fine.
Or are you compiling with -O0 or something similar?

I think it makes sense to replace Leap with Tumbleweed.

IIRC we wanted to avoid rolling releases in the CI since this would rather force us to deal with distro bugs on a regular basis instead of focusing on bugs in QEMU.

 Thomas



Reply via email to