Message ID | 20230110160233.339771-7-pbonzini@redhat.com |
---|---|
State | New |
Headers | show |
Series | [PULL,01/29] configure: fix GLIB_VERSION for cross-compilation | expand |
On 10/1/23 17:02, Paolo Bonzini wrote: > From: Eric Auger <eric.auger@redhat.com> > > To avoid compilation errors when -Werror=maybe-uninitialized is used, > replace 'case 3' by 'default'. > > Otherwise we get: > > ../target/i386/ops_sse.h: In function ‘helper_vpermdq_ymm’: > ../target/i386/ops_sse.h:2495:13: error: ‘r3’ may be used Thanksfully these odd chars ^^^^^^^^ ... > Signed-off-by: Eric Auger <eric.auger@redhat.com> > Suggested-by: Stefan Weil <sw@weilnetz.de> > Fixes: 790684776861 ("target/i386: reimplement 0x0f 0x3a, add AVX") > Message-Id: <20221221163652.1239362-1-eric.auger@redhat.com> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> > --- > target/i386/ops_sse.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) ... aren't included in your staged commits in https://gitlab.com/bonzini/qemu.git tags/for-upstream: commit c0a6665c3c4d63b113ab31c624c53d4a32de2926 Author: Eric Auger <eric.auger@redhat.com> Date: Thu Dec 22 15:01:58 2022 +0100 ../target/i386/ops_sse.h: In function ‘helper_vpermdq_ymm’: ../target/i386/ops_sse.h:2495:13: error: ‘r3’ may be used uninitialized in this function [-Werror=maybe-uninitialized] 2495 | d->Q(3) = r3; | ~~~~~~~~^~~~ So not a blocker. Maybe a git-send-email issue?
diff --git a/target/i386/ops_sse.h b/target/i386/ops_sse.h index 3cbc36a59d1a..c442c8c10cdc 100644 --- a/target/i386/ops_sse.h +++ b/target/i386/ops_sse.h @@ -2466,7 +2466,7 @@ void helper_vpermdq_ymm(Reg *d, Reg *v, Reg *s, uint32_t order) r0 = s->Q(0); r1 = s->Q(1); break; - case 3: + default: r0 = s->Q(2); r1 = s->Q(3); break; @@ -2484,7 +2484,7 @@ void helper_vpermdq_ymm(Reg *d, Reg *v, Reg *s, uint32_t order) r2 = s->Q(0); r3 = s->Q(1); break; - case 3: + default: r2 = s->Q(2); r3 = s->Q(3); break;