From owner-svn-src-head@FreeBSD.ORG Mon May 2 10:43:44 2011 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1233) id 12A931065670; Mon, 2 May 2011 10:43:44 +0000 (UTC) Date: Mon, 2 May 2011 10:43:44 +0000 From: Alexander Best To: Bruce Cran Message-ID: <20110502104344.GA85667@freebsd.org> References: <201105021035.p42AZSfp034883@svn.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201105021035.p42AZSfp034883@svn.freebsd.org> Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r221320 - head/sys/conf X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 02 May 2011 10:43:44 -0000 On Mon May 2 11, Bruce Cran wrote: > Author: brucec > Date: Mon May 2 10:35:27 2011 > New Revision: 221320 > URL: http://svn.freebsd.org/changeset/base/221320 > > Log: > Add -Wmissing-include-dirs to CWARNFLAGS, so tinderbox will punish those > developers committing new code with broken include directories. Bruce already committed r220712 in order to make the current tree work with -Wmissing-include-dirs. So hopefully tinderbox will not complain for any ARCH. Please also note that currently -Wmissing-include-dirs is a noop in clang. Once it gets implemented we will hopefully benefit from it via the vendor import. Thanks again Bruce for taking care! :) > Fix a few whitespace issues. > Improve a couple of comments. > -W is now deprecated and is referred to as -Wextra (see gcc(1)). > > Submitted by: arundel > > Modified: > head/sys/conf/kern.mk > > Modified: head/sys/conf/kern.mk > ============================================================================== > --- head/sys/conf/kern.mk Mon May 2 09:47:13 2011 (r221319) > +++ head/sys/conf/kern.mk Mon May 2 10:35:27 2011 (r221320) > @@ -3,24 +3,25 @@ > # > # Warning flags for compiling the kernel and components of the kernel. > # > -# Note that the newly added -Wcast-qual is responsible for generating > +# Note that the newly added -Wcast-qual is responsible for generating > # most of the remaining warnings. Warnings introduced with -Wall will > # also pop up, but are easier to fix. > CWARNFLAGS?= -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes \ > -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual \ > - -Wundef -Wno-pointer-sign -fformat-extensions > + -Wundef -Wno-pointer-sign -fformat-extensions \ > + -Wmissing-include-dirs > # > # The following flags are next up for working on: > -# -W > +# -Wextra > > # > -# On the i386, do not align the stack to 16-byte boundaries. Otherwise GCC > -# 2.95 adds code to the entry and exit point of every function to align the > +# On i386, do not align the stack to 16-byte boundaries. Otherwise GCC 2.95 > +# and above adds code to the entry and exit point of every function to align the > # stack to 16-byte boundaries -- thus wasting approximately 12 bytes of stack > -# per function call. While the 16-byte alignment may benefit micro benchmarks, > +# per function call. While the 16-byte alignment may benefit micro benchmarks, > # it is probably an overall loss as it makes the code bigger (less efficient > # use of code cache tag lines) and uses more stack (less efficient use of data > -# cache tag lines). Explicitly prohibit the use of SSE and other SIMD > +# cache tag lines). Explicitly prohibit the use of FPU, SSE and other SIMD > # operations inside the kernel itself. These operations are exclusively > # reserved for user applications. > # > @@ -35,6 +36,7 @@ INLINE_LIMIT?= 8000 > .if ${MACHINE_CPUARCH} == "arm" > INLINE_LIMIT?= 8000 > .endif > + > # > # For IA-64, we use r13 for the kernel globals pointer and we only use > # a very small subset of float registers for integer divides. > @@ -98,7 +100,7 @@ INLINE_LIMIT?= 8000 > CFLAGS+= -ffreestanding > > # > -# GCC SSP support. > +# GCC SSP support > # > .if ${MK_SSP} != "no" && ${MACHINE_CPUARCH} != "ia64" && \ > ${MACHINE_CPUARCH} != "arm" && ${MACHINE_CPUARCH} != "mips" > @@ -106,9 +108,8 @@ CFLAGS+= -fstack-protector > .endif > > # > -# Enable CTF conversation on request. > +# Enable CTF conversation on request > # > .if defined(WITH_CTF) > .undef NO_CTF > .endif > - -- a13x