[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/7] clean build - eliminate warnings
From: |
Laurent Desnogues |
Subject: |
Re: [Qemu-devel] [PATCH 0/7] clean build - eliminate warnings |
Date: |
Sat, 21 Feb 2009 20:43:24 +0100 |
On Sat, Feb 21, 2009 at 8:00 PM, Jan Kiszka <address@hidden> wrote:
> When working on larger or intrusive changes like the monitor rework, the
> 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 cosmetic
> 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)
This means that after applying your patch there should be no more
warning for the ARM target?
On my machine (x86_64, gcc 4.1.2), I still get these:
CC arm-softmmu/neon_helper.o
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c: In
function ‘helper_neon_rshl_s8’:
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469:
warning: ‘vdest.v1’ is used uninitialized in this function
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469:
warning: ‘vdest.v2’ is used uninitialized in this function
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469:
warning: ‘vdest.v3’ is used uninitialized in this function
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:469:
warning: ‘vdest.v4’ is used uninitialized in this function
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c: In
function ‘helper_neon_rshl_s16’:
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:470:
warning: ‘vdest.v1’ is used uninitialized in this function
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:470:
warning: ‘vdest.v2’ is used uninitialized in this function
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c: In
function ‘helper_neon_rshl_s32’:
/home/ldesnogu/work/Emu/qemu/svn-ref/target-arm/neon_helper.c:471:
warning: ‘vdest.v1’ is used uninitialized in this function
Note a patch has been proposed in the past (by Aurélien IIRC).
Laurent
- [Qemu-devel] [PATCH 7/7] clean build: Fix remaining sh4 warnings, (continued)
- [Qemu-devel] [PATCH 7/7] clean build: Fix remaining sh4 warnings, Jan Kiszka, 2009/02/21
- [Qemu-devel] [PATCH 3/7] arm: Fix gic_irq_state.level bitfield type, Jan Kiszka, 2009/02/21
- [Qemu-devel] [PATCH 4/7] clean build: Fix arm build warnings, Jan Kiszka, 2009/02/21
- [Qemu-devel] [PATCH 5/7] clean build: Fix remaining cris warnings, Jan Kiszka, 2009/02/21
- Re: [Qemu-devel] [PATCH 5/7] clean build: Fix remaining cris warnings, Paul Brook, 2009/02/21
Re: [Qemu-devel] [PATCH 0/7] clean build - eliminate warnings,
Laurent Desnogues <=