From owner-freebsd-ia64@FreeBSD.ORG  Thu Mar 18 00:41:33 2004
Return-Path: <owner-freebsd-ia64@FreeBSD.ORG>
Delivered-To: freebsd-ia64@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id CF17516A4CE; Thu, 18 Mar 2004 00:41:33 -0800 (PST)
Received: from ns1.xcllnt.net (209-128-86-226.bayarea.net [209.128.86.226])
	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 8CA7343D3F; Thu, 18 Mar 2004 00:41:33 -0800 (PST)
	(envelope-from marcel@xcllnt.net)
Received: from dhcp01.pn.xcllnt.net (dhcp01.pn.xcllnt.net [192.168.4.201])
	by ns1.xcllnt.net (8.12.11/8.12.11) with ESMTP id i2I8fTp3058194;
	Thu, 18 Mar 2004 00:41:29 -0800 (PST)
	(envelope-from marcel@piii.pn.xcllnt.net)
Received: from dhcp01.pn.xcllnt.net (localhost [127.0.0.1])
	i2I8fTqw059212;	Thu, 18 Mar 2004 00:41:29 -0800 (PST)
	(envelope-from marcel@dhcp01.pn.xcllnt.net)
Received: (from marcel@localhost)
	by dhcp01.pn.xcllnt.net (8.12.11/8.12.11/Submit) id i2I8fSRv059211;
	Thu, 18 Mar 2004 00:41:28 -0800 (PST)
	(envelope-from marcel)
Date: Thu, 18 Mar 2004 00:41:28 -0800
From: Marcel Moolenaar <marcel@xcllnt.net>
To: Ruslan Ermilov <ru@FreeBSD.org>
Message-ID: <20040318084128.GA59199@dhcp01.pn.xcllnt.net>
References: <20040317040254.386947303A@freebsd-current.sentex.ca>
	<xzpad2fhixn.fsf@dwp.des.no> <20040317113931.GF49920@ip.net.ua>
	<20040317200307.GA46126@ip.net.ua>
	<20040317205104.GA29066@dhcp01.pn.xcllnt.net>
	<20040317213014.GB46747@ip.net.ua>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20040317213014.GB46747@ip.net.ua>
User-Agent: Mutt/1.4.2.1i
cc: FreeBSD Tinderbox <tinderbox@FreeBSD.org>
cc: current@FreeBSD.org
cc: ia64@FreeBSD.org
Subject: Re: [current tinderbox] failure on ia64/ia64
X-BeenThere: freebsd-ia64@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: Porting FreeBSD to the IA-64 <freebsd-ia64.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ia64>,
	<mailto:freebsd-ia64-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-ia64>
List-Post: <mailto:freebsd-ia64@freebsd.org>
List-Help: <mailto:freebsd-ia64-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-ia64>,
	<mailto:freebsd-ia64-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 18 Mar 2004 08:41:34 -0000

On Wed, Mar 17, 2004 at 11:30:14PM +0200, Ruslan Ermilov wrote:
> On Wed, Mar 17, 2004 at 12:51:04PM -0800, Marcel Moolenaar wrote:
> > On Wed, Mar 17, 2004 at 10:03:07PM +0200, Ruslan Ermilov wrote:
> > > > > 
> > > > >         for(i = 0; i < SOUND_MIXER_NRDEVICES; i++) {
> > > > >                 if ((1 << i) & mask) {
> > > > >                         v |= mt[i].iselect;
> > > > >                 }
> > > > >         }
> > > > > 
> > > > 
> > > Actually, this is easily reproduceable by compiling the corresponding
> > > module on pluto1 with CFLAGS="-O2 -pipe" set in environment.  There's
> > > clearly a bug in gcc(1), as:
> > 
> > What happens if you replace the for-loop with an equivalent while-loop?
> > 
> Doesn't make any difference.  sparc64 and amd64 do not exhibit this problem.
> (Yes, I tested this on amd64 as well, skipping the final ld(1) step of the
> module's build.)

The source code appears to be fishy. The variable v is never used.
It's only defined. If you change the "return mask" to a "return v",
the warning goes away.

-- 
 Marcel Moolenaar	  USPA: A-39004		 marcel@xcllnt.net