Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 17 Mar 2004 22:03:07 +0200
From:      Ruslan Ermilov <ru@freebsd.org>
To:        Dag-Erling Sm?rgrav <des@des.no>
Cc:        FreeBSD Tinderbox <tinderbox@freebsd.org>
Subject:   Re: [current tinderbox] failure on ia64/ia64
Message-ID:  <20040317200307.GA46126@ip.net.ua>
In-Reply-To: <20040317113931.GF49920@ip.net.ua>
References:  <20040317040254.386947303A@freebsd-current.sentex.ca> <xzpad2fhixn.fsf@dwp.des.no> <20040317113931.GF49920@ip.net.ua>

next in thread | previous in thread | raw e-mail | index | archive | help

--OXfL5xGRrasGEqWY
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Mar 17, 2004 at 01:39:31PM +0200, Ruslan Ermilov wrote:
> On Wed, Mar 17, 2004 at 11:22:28AM +0100, Dag-Erling Sm?rgrav wrote:
> > FreeBSD Tinderbox <tinderbox@freebsd.org> writes:
> > > /other/tinderbox/CURRENT/ia64/ia64/src/sys/dev/sound/pci/vibes.c: In =
function `sv_mix_setrecsrc':
> > > /other/tinderbox/CURRENT/ia64/ia64/src/sys/dev/sound/pci/vibes.c:541:=
 warning: `i' might be used uninitialized in this function
> > > *** Error code 1
> >=20
> > Now that is wierd...  and I can't reproduce it.  There is no way i can
> > be used uninitialized, it is not referenced anywhere outside this
> > loop:
> >=20
> >         for(i =3D 0; i < SOUND_MIXER_NRDEVICES; i++) {
> >                 if ((1 << i) & mask) {
> >                         v |=3D mt[i].iselect;
> >                 }
> >         }
> >=20
> > If the problem persists, we'll have to conclude that gcc is broken
> > somehow.
> >=20
> I cannot reproduce it either, but perhaps this is the same reason why
> we still have -Wno-uninitialized in <bsd.sys.mk> for normal WARNS
> levels.
>=20
Actually, this is easily reproduceable by compiling the corresponding
module on pluto1 with CFLAGS=3D"-O2 -pipe" set in environment.  There's
clearly a bug in gcc(1), as:

- substituting 2 instead of SOUND_MIXER_NRDEVICES in the line below
  makes it compile without a warning;

        for(i =3D 0; i < SOUND_MIXER_NRDEVICES; i++) {

- Bumping it to 3 or greater gives this mis-warning.


Cheers,
--=20
Ruslan Ermilov
FreeBSD committer
ru@FreeBSD.org

--OXfL5xGRrasGEqWY
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (FreeBSD)

iD8DBQFAWK77Ukv4P6juNwoRAp5HAKCHKkiIQ3XUgviMgWrV4QgePGJ3QwCcCyLu
XPQREVENbvqQNh8T+alZYc8=
=pmTg
-----END PGP SIGNATURE-----

--OXfL5xGRrasGEqWY--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040317200307.GA46126>