Re: [HEADS-UP] Problem with clang in 9-stable [was: r268244 (stable/9) seems to break "sysctl hw.ncpu"]

2014-07-05 Thread David Chisnall
On 5 Jul 2014, at 14:07, Dimitry Andric wrote: > Interestingly, -Wno-uninitialized has been in bsd.sys.mk since r76861, > and the accompanying comment ("XXX Delete -Wuninitialized by default for > now -- the compiler doesn't always get it right") has never been > changed. :-) > > It is probably

Re: [HEADS-UP] Problem with clang in 9-stable [was: r268244 (stable/9) seems to break "sysctl hw.ncpu"]

2014-07-05 Thread Dimitry Andric
On 05 Jul 2014, at 14:49, David Chisnall wrote: > On 4 Jul 2014, at 19:18, David Wolfskill wrote: > >> clang -O2 -pipe -std=gnu99 -Qunused-arguments -fstack-protector >> -Wsystem-headers -Werror -Wall -Wno-format-y2k -W -Wno-unused-parameter >> -Wstrict-prototypes -Wmissing-prototypes -Wpoin

Re: [HEADS-UP] Problem with clang in 9-stable [was: r268244 (stable/9) seems to break "sysctl hw.ncpu"]

2014-07-05 Thread David Wolfskill
On Sat, Jul 05, 2014 at 01:49:44PM +0100, David Chisnall wrote: > On 4 Jul 2014, at 19:18, David Wolfskill wrote: > > > clang -O2 -pipe -std=gnu99 -Qunused-arguments -fstack-protector > > -Wsystem-headers -Werror -Wall -Wno-format-y2k -W -Wno-unused-parameter > > -Wstrict-prototypes -Wmissing

Re: [HEADS-UP] Problem with clang in 9-stable [was: r268244 (stable/9) seems to break "sysctl hw.ncpu"]

2014-07-04 Thread Hans Petter Selasky
Hi, I've reverted the MFC of r267960: http://svnweb.freebsd.org/changeset/base/268263 I did some checking and the patch makes access to a non-initialized variable, due to other missing patches prior to mine, probably making compiler differences show up on my side. It appears the patch cannot b

Re: [HEADS-UP] Problem with clang in 9-stable [was: r268244 (stable/9) seems to break "sysctl hw.ncpu"]

2014-07-04 Thread David Wolfskill
On Fri, Jul 04, 2014 at 08:09:21PM +0200, Hans Petter Selasky wrote: > Hi, > > Can you try to reproduce this: > > Checkout 9-stable's src/sbin/sysctl only. > > cd /usr/9-stable/src/sbin/sysctl > > env CC=clang make clean all > ./sysctl -n hw.ncpu > > Then: > > env CC=gcc make clean all > ./sy

[HEADS-UP] Problem with clang in 9-stable [was: r268244 (stable/9) seems to break "sysctl hw.ncpu"]

2014-07-04 Thread Hans Petter Selasky
Hi, Can you try to reproduce this: Checkout 9-stable's src/sbin/sysctl only. cd /usr/9-stable/src/sbin/sysctl env CC=clang make clean all ./sysctl -n hw.ncpu Then: env CC=gcc make clean all ./sysctl -n hw.ncpu clang --version FreeBSD clang version 3.2 (tags/RELEASE_32/final 170710) 2012122