Re: [PATCH 4/4] target/ppc: fix warning with clang-15

2023-02-15 Thread Pierrick Bouvier
On 2/14/23 19:10, Richard Henderson wrote: On 2/13/23 06:13, Pierrick Bouvier wrote: When compiling for windows-arm64 using clang-15, it reports a sometimes uninitialized variable. This seems to be a false positive, as a default case guards switch expressions, preventing to return an uninitializ

Re: [PATCH 4/4] target/ppc: fix warning with clang-15

2023-02-14 Thread Richard Henderson
On 2/13/23 06:13, Pierrick Bouvier wrote: When compiling for windows-arm64 using clang-15, it reports a sometimes uninitialized variable. This seems to be a false positive, as a default case guards switch expressions, preventing to return an uninitialized value, but clang seems unhappy with asser

Re: [PATCH 4/4] target/ppc: fix warning with clang-15

2023-02-14 Thread Philippe Mathieu-Daudé
On 13/2/23 17:13, Pierrick Bouvier wrote: When compiling for windows-arm64 using clang-15, it reports a sometimes uninitialized variable. This seems to be a false positive, as a default case guards switch expressions, preventing to return an uninitialized value, but clang seems unhappy with asser

Re: [PATCH 4/4] target/ppc: fix warning with clang-15

2023-02-14 Thread Pierrick Bouvier
On 2/14/23 08:14, Philippe Mathieu-Daudé wrote: On 13/2/23 17:13, Pierrick Bouvier wrote: When compiling for windows-arm64 using clang-15, it reports a sometimes uninitialized variable. This seems to be a false positive, as a default case guards switch expressions, preventing to return an uninit

[PATCH 4/4] target/ppc: fix warning with clang-15

2023-02-13 Thread Pierrick Bouvier
When compiling for windows-arm64 using clang-15, it reports a sometimes uninitialized variable. This seems to be a false positive, as a default case guards switch expressions, preventing to return an uninitialized value, but clang seems unhappy with assert definition. Setting the rnd variable to z

Re: [PATCH 4/4] target/ppc: fix warning with clang-15

2023-02-13 Thread Cédric Le Goater
On 2/13/23 17:13, Pierrick Bouvier wrote: When compiling for windows-arm64 using clang-15, it reports a sometimes uninitialized variable. This seems to be a false positive, as a default case guards switch expressions, preventing to return an uninitialized value, but clang seems unhappy with asser