From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Lay9w-0003s3-KH for qemu-devel@nongnu.org; Sat, 21 Feb 2009 15:09:12 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Lay9w-0003rT-6w for qemu-devel@nongnu.org; Sat, 21 Feb 2009 15:09:12 -0500 Received: from [199.232.76.173] (port=34122 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Lay9v-0003rH-Pt for qemu-devel@nongnu.org; Sat, 21 Feb 2009 15:09:11 -0500 Received: from fmmailgate01.web.de ([217.72.192.221]:46898) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1Lay9u-00057i-Pp for qemu-devel@nongnu.org; Sat, 21 Feb 2009 15:09:11 -0500 Received: from smtp08.web.de (fmsmtp08.dlan.cinetic.de [172.20.5.216]) by fmmailgate01.web.de (Postfix) with ESMTP id E30C1FDAA9AA for ; Sat, 21 Feb 2009 21:09:09 +0100 (CET) Received: from [88.65.43.110] (helo=[192.168.1.198]) by smtp08.web.de with asmtp (TLSv1:AES256-SHA:256) (WEB.DE 4.110 #277) id 1Lay9t-0003tR-00 for qemu-devel@nongnu.org; Sat, 21 Feb 2009 21:09:09 +0100 Message-ID: <49A05F5D.6070903@web.de> Date: Sat, 21 Feb 2009 21:09:01 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <20090221190054.12485.30486.stgit@mchn012c.ww002.siemens.net> <761ea48b0902211143x3f791ca8k5e6d0709076ce009@mail.gmail.com> In-Reply-To: <761ea48b0902211143x3f791ca8k5e6d0709076ce009@mail.gmail.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig2DEC613A7207F467484E3AC3" Sender: jan.kiszka@web.de Subject: [Qemu-devel] Re: [PATCH 0/7] clean build - eliminate warnings Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig2DEC613A7207F467484E3AC3 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Laurent Desnogues wrote: > On Sat, Feb 21, 2009 at 8:00 PM, Jan Kiszka wrote: >> When working on larger or intrusive changes like the monitor rework, t= he >> number of warnings a normal build generates (here: x86-64 host, gcc 4.= 3) >> is still too high. And sometimes these warnings are not just of cosmet= ic >> nature, see (reposted) patch 3. >> >> This series reduces the number of warnings significantly, still not to= >> zero (someone would have to look into the NetWinder stuff), but almost= : >> >> Warning summary for 2009-02-21 (changes since 2009-02-21-base) >> generic 0 (-1) >> softmmu 0 (-39) >> x86 0 (0) >> arm 0 (-10) >=20 > This means that after applying your patch there should be no more > warning for the ARM target? At least for softmmu, at least with my compiler (depending on the precise version / distro patches, you may have different warnings enabled by default): yes. >=20 > On my machine (x86_64, gcc 4.1.2), I still get these: >=20 > CC arm-softmmu/neon_helper.o > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c: In > function =91helper_neon_rshl_s8=92: > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469: > warning: =91vdest.v1=92 is used uninitialized in this function > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469: > warning: =91vdest.v2=92 is used uninitialized in this function > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469: > warning: =91vdest.v3=92 is used uninitialized in this function > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469: > warning: =91vdest.v4=92 is used uninitialized in this function > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c: In > function =91helper_neon_rshl_s16=92: > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:470: > warning: =91vdest.v1=92 is used uninitialized in this function > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:470: > warning: =91vdest.v2=92 is used uninitialized in this function > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c: In > function =91helper_neon_rshl_s32=92: > /home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:471: > warning: =91vdest.v1=92 is used uninitialized in this function Has this been identified as a real issue or just compiler blindness (my series contains one "fix" for such blindness, see cris patch)? I'm currently a bit lost in those macros... >=20 > Note a patch has been proposed in the past (by Aur=E9lien IIRC). Do you have a reference at hand? Jan --------------enig2DEC613A7207F467484E3AC3 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.9 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org iEYEARECAAYFAkmgX2UACgkQniDOoMHTA+kVeQCfR38YzyH4wE6PVnDKKQdOXezA 908AnjIuyhqKA7RP59ys1HmeYsvcTBTA =4Djc -----END PGP SIGNATURE----- --------------enig2DEC613A7207F467484E3AC3--