From owner-cvs-src@FreeBSD.ORG  Wed Jul 30 20:06:23 2003
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP
	id 87BF437B401; Wed, 30 Jul 2003 20:06:23 -0700 (PDT)
Received: from HAL9000.homeunix.com (ip114.bella-vista.sfo.interquest.net
	[66.199.86.114])	by mx1.FreeBSD.org (Postfix) with ESMTP
	id 1042843FAF; Wed, 30 Jul 2003 20:06:22 -0700 (PDT)
	(envelope-from das@FreeBSD.ORG)
Received: from HAL9000.homeunix.com (localhost [127.0.0.1])
	by HAL9000.homeunix.com (8.12.9/8.12.9) with ESMTP id h6V36JBG007877;
	Wed, 30 Jul 2003 20:06:19 -0700 (PDT)
	(envelope-from das@FreeBSD.ORG)
Received: (from das@localhost)
	by HAL9000.homeunix.com (8.12.9/8.12.9/Submit) id h6V36IWI007876;
	Wed, 30 Jul 2003 20:06:18 -0700 (PDT)
	(envelope-from das@FreeBSD.ORG)
Date: Wed, 30 Jul 2003 20:06:18 -0700
From: David Schultz <das@FreeBSD.ORG>
To: Scott Long <scottl@FreeBSD.ORG>
Message-ID: <20030731030618.GA7795@HAL9000.homeunix.com>
Mail-Followup-To: Scott Long <scottl@FreeBSD.org>,
	Ruslan Ermilov <ru@FreeBSD.org>, src-committers@FreeBSD.org,
	cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
References: <200307301446.h6UEklCE044311@repoman.freebsd.org>
	<3F27EFCF.4070605@freebsd.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <3F27EFCF.4070605@freebsd.org>
cc: cvs-src@FreeBSD.ORG
cc: src-committers@FreeBSD.ORG
cc: Ruslan Ermilov <ru@FreeBSD.ORG>
cc: cvs-all@FreeBSD.ORG
Subject: Re: cvs commit: src/release Makefile
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Thu, 31 Jul 2003 03:06:24 -0000

On Wed, Jul 30, 2003, Scott Long wrote:
> Ruslan Ermilov wrote:
> >ru          2003/07/30 07:46:46 PDT
> >
> >  FreeBSD src repository
> >
> >  Modified files:
> >    release              Makefile 
> >  Log:
> >  The -Os kernel build with GCC 3.3.1 also requires -fno-strict-aliasing
> >  to survive.
> >  
> >  Discussed with: bde
> >  Tested on:      i386 and pc98
> >  
> >  Revision  Changes    Path
> >  1.806     +2 -1      src/release/Makefile
> >
> 
> Thanks for doing this.  However, I think that Peter was looking at doing
> this globally in /sys/conf, not just the release.  The reasoning is that
> it might not be safe to use -fstrict-aliasing in any context right now
> until we understand the actual impact of those warnings.  I'd also
> prefer if it was done there and not special-cased for the release.

It has never been safe to compile the kernel with
-fstrict-aliasing (or consequently -O2) in gcc3.
Peter fixed a place where the warning was bogus,
not a place where the kernel violated C's aliasing
rules in a way that would cause gcc to make incorrect
optimizations.