From owner-cvs-src@FreeBSD.ORG  Thu Nov 22 23:21:13 2007
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 6941716A419;
	Thu, 22 Nov 2007 23:21:13 +0000 (UTC) (envelope-from jb@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id 659F513C458;
	Thu, 22 Nov 2007 23:21:13 +0000 (UTC) (envelope-from jb@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id lAMNLD12082679;
	Thu, 22 Nov 2007 23:21:13 GMT (envelope-from jb@repoman.freebsd.org)
Received: (from jb@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id lAMNLD4K082678;
	Thu, 22 Nov 2007 23:21:13 GMT (envelope-from jb)
Message-Id: <200711222321.lAMNLD4K082678@repoman.freebsd.org>
From: John Birrell <jb@FreeBSD.org>
Date: Thu, 22 Nov 2007 23:21:13 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/share/mk bsd.sys.mk sys.mk
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.5
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, 22 Nov 2007 23:21:13 -0000

jb          2007-11-22 23:21:13 UTC

  FreeBSD src repository

  Modified files:
    share/mk             bsd.sys.mk sys.mk 
  Log:
  Re-enable -Werror again.
  
  This time, change the default CFLAGS to match the simple defaults that
  the tinderboxes use. That is, don't use -fno-strict-aliasing by default.
  
  My last attempt to re-anable -Werror gave me a lesson in what strict
  aliasing is all about. There was code in libthr that wasn't 64-bit clean.
  The default use of -fno-strict-aliasing hid that.
  
  By using -fno-strict-aliasing by default we were choosing to ignore
  problems in code which had the potential to shoot ourselves in the
  foot. Sometimes it would be the 64-bit foot. I have both feet. The left
  ones are 32 bits and the right ones are 64 bits. Don't ask about my
  endian orientation. :-)
  
  The -fno-strict-aliasing compiler arg can still be used if NO_STRING_ALIASING
  is define in make.
  
  We are early in the FreeBSD 8 development, so we have the opportunity to
  wait and see if this works for us. I am sure that people will complain.
  We can easily revert this. All I ask is that we take sides: clean code or
  not. YMMV.
  
  Note that by using -fno-strict-aliasing the build won't actually break.
  Only where WARNS is set (and -Werror is used) will a compiler warning break
  the build. The use of WARNS levels implies (to me at least) that the
  developer has taken some care to make the code pass basic checks. This
  commit makes those checks just a little bit more strict.
  
  Revision  Changes    Path
  1.44      +0 -2      src/share/mk/bsd.sys.mk
  1.96      +5 -2      src/share/mk/sys.mk