Re: 8.1-rc0 testfloat fails to compile

[Thomas Huth <thuth@redhat.com>]

On 21/07/2023 09.03, Olaf Hering wrote:
> Fri, 21 Jul 2023 08:54:21 +0200 Thomas Huth <thuth@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