Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 27 May 2011 18:29:06 +0000
From:      Alexander Best <arundel@freebsd.org>
To:        Warner Losh <imp@bsdimp.com>
Cc:        freebsd-hackers@FreeBSD.ORG, freebsd-toolchain@FreeBSD.ORG
Subject:   Re: [rfc] a few kern.mk and bsd.sys.mk related changes
Message-ID:  <20110527182906.GA31871@freebsd.org>
In-Reply-To: <20110527181459.GA29908@freebsd.org>
References:  <20110527115147.GA73802@freebsd.org> <3BF63174-1B29-4A4D-96DD-3ED65ED96EAC@bsdimp.com> <20110527181459.GA29908@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri May 27 11, Alexander Best wrote:
> On Fri May 27 11, Warner Losh wrote:
> > These look generally good.  Just one thing I had a question on:
> > 
> >  #
> > +# Enable FreeBSD kernel-specific printf format specifiers. Also instruct gcc to
> > +# enable some diagnostics, which make it easier to pinpoint tinderbox failures.
> > +CFLAGS+=	-fformat-extensions -fdiagnostics-show-option
> > +
> > 
> > Does this put sand in the gears of clang support?

i've improved the above comment so as to note that clang won't be affected by
-fdiagnostics-show-option. the new patch is available here:

http://people.freebsd.org/~arundel/patches/kern.mk-bsy.sys.mk.patch

another "issue" i've come across is the fact that on amd64, -O2 is always being
set, no matter, if debugging is enabled or disabled. i think amd64 should also
(just like all the other archs) be using -O, when debugging was enabled.

changing this is quite easy. so if people think this is something worth
changing i can integrate it into my existing patch.

cheers.
alex

> 
> i don't think so. the code that features -fformat-extensions support is freebsd
> specific and gets plugged into the vendor code during every new import. dim@
> and rdivacky@ have been discussing the possibility of getting this committed
> upstream as something like -ffreebsd-extensions, but they haven't decided, yet.
> 
> regarding -fdiagnostics-show-option: this is intended purely for gcc, since it
> is disabled per default. under clang the flag exists, too, but it is enabled by
> default. so specifying it under clang is pretty much a nop.
> 
> cheers.
> alex
> 
> > 
> > Warner
> > 
> > 
> > On May 27, 2011, at 5:51 AM, Alexander Best wrote:
> > 
> > > hi there,
> > > 
> > > i would like to hear people's opinion regarding the following patch. it
> > > features the following semantic changes:
> > > 
> > > 1) -W is now called -Wextra
> > > 2) move -fformat-extensions and -fdiagnostics-show-option out of CWARNFLAGS,
> > >   since they aren't really warnings. this will now let anybody use custom
> > >   CWARNFLAGS without having to remember to add -fformat-extensions in order
> > >   not to break buildworld.
> > > 3) rearrange "\" characters, as to avoid double spaces
> > > 4) when building kernel modules, -fstack-protector was added to CFLAGS twice.
> > >   this should now be solved.
> > > 
> > > cheers.
> > > alex
> > > 
> > > -- 
> > > a13x
> > > <kern.mk-bsy.sys.mk.patch>_______________________________________________
> > > freebsd-toolchain@freebsd.org mailing list
> > > http://lists.freebsd.org/mailman/listinfo/freebsd-toolchain
> > > To unsubscribe, send any mail to "freebsd-toolchain-unsubscribe@freebsd.org"
> 
> -- 
> a13x

-- 
a13x



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