From owner-svn-src-all@FreeBSD.ORG Thu Jan 1 03:52:28 2015 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id C6950FEE; Thu, 1 Jan 2015 03:52:28 +0000 (UTC) Received: from tensor.andric.com (unknown [IPv6:2001:7b8:3a7:1:2d0:b7ff:fea0:8c26]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (Client CN "tensor.andric.com", Issuer "COMODO RSA Domain Validation Secure Server CA" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 6B82864CDE; Thu, 1 Jan 2015 03:52:28 +0000 (UTC) Received: from [IPv6:2001:7b8:3a7::28eb:6a07:fcba:ed4a] (unknown [IPv6:2001:7b8:3a7:0:28eb:6a07:fcba:ed4a]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by tensor.andric.com (Postfix) with ESMTPSA id 9F05FB80A; Thu, 1 Jan 2015 04:52:22 +0100 (CET) Subject: Re: svn commit: r276488 - in head: share/mk sys/arm/broadcom/bcm2835 sys/conf Mime-Version: 1.0 (Mac OS X Mail 8.1 \(1993\)) Content-Type: multipart/signed; boundary="Apple-Mail=_ED03ED01-C25D-4F80-8742-DB50BD9CB951"; protocol="application/pgp-signature"; micalg=pgp-sha1 X-Pgp-Agent: GPGMail 2.5b4 From: Dimitry Andric In-Reply-To: <201501010200.t01205Ge007058@svn.freebsd.org> Date: Thu, 1 Jan 2015 04:52:16 +0100 Message-Id: <760D3407-921D-4249-9DC2-74610FA46920@FreeBSD.org> References: <201501010200.t01205Ge007058@svn.freebsd.org> To: Warner Losh X-Mailer: Apple Mail (2.1993) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 01 Jan 2015 03:52:29 -0000 --Apple-Mail=_ED03ED01-C25D-4F80-8742-DB50BD9CB951 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=us-ascii On 01 Jan 2015, at 03:00, Warner Losh wrote: >=20 > Author: imp > Date: Thu Jan 1 02:00:04 2015 > New Revision: 276488 > URL: https://svnweb.freebsd.org/changeset/base/276488 >=20 > Log: > Fix module builds on arm (and maybe others) by turning off a whole > raft of new warnings that appear to be on by default in clang 3.5.0. > Fix RPI-B build issues with new clang not liking the ability to pass > arbitrary flags to as, since some flags are more arbitrary (and thus > verboten) than others. >=20 > These warnings should be actually fixed in the code, but this is a > band-aide to get things (almost) building again. >=20 > Modified: > head/share/mk/bsd.kmod.mk > head/share/mk/bsd.sys.mk > head/sys/arm/broadcom/bcm2835/std.bcm2835 > head/sys/conf/kern.mk >=20 > Modified: head/share/mk/bsd.kmod.mk > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- head/share/mk/bsd.kmod.mk Thu Jan 1 01:43:00 2015 = (r276487) > +++ head/share/mk/bsd.kmod.mk Thu Jan 1 02:00:04 2015 = (r276488) > @@ -13,5 +13,3 @@ SYSDIR=3D ${_dir} > .endif >=20 > .include "${SYSDIR}/conf/kmod.mk" > - > -.include >=20 > Modified: head/share/mk/bsd.sys.mk > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- head/share/mk/bsd.sys.mk Thu Jan 1 01:43:00 2015 = (r276487) > +++ head/share/mk/bsd.sys.mk Thu Jan 1 02:00:04 2015 = (r276488) > @@ -110,11 +110,7 @@ CWARNFLAGS+=3D -Wno-format > CWARNFLAGS+=3D -Wno-unknown-pragmas > .endif # IGNORE_PRAGMA >=20 > -.if ${COMPILER_TYPE} =3D=3D "clang" > -# Would love to do this unconditionally, but can't due to its use in > -# kernel build coupled with CFLAGS.${TARGET} feature > CLANG_NO_IAS=3D -no-integrated-as > -.endif > CLANG_OPT_SMALL=3D -mstack-alignment=3D8 -mllvm -inline-threshold=3D3\ > -mllvm -simplifycfg-dup-ret -mllvm -enable-gvn=3Dfalse > CFLAGS.clang+=3D -Qunused-arguments This should not be removed from bsd.sys.mk, since other things depend on it, are you sure that you want this? :) As far as I know, there are still a few places in the tree that use the CLANG_NO_IAS macro. It would be nice if those all went away, though... > Modified: head/sys/arm/broadcom/bcm2835/std.bcm2835 > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- head/sys/arm/broadcom/bcm2835/std.bcm2835 Thu Jan 1 01:43:00 2015 = (r276487) > +++ head/sys/arm/broadcom/bcm2835/std.bcm2835 Thu Jan 1 02:00:04 2015 = (r276488) > @@ -2,7 +2,7 @@ >=20 > machine arm armv6 > cpu CPU_ARM1176 > -makeoptions CONF_CFLAGS=3D"-mcpu=3Darm1176jzf-s = -Wa,-mcpu=3Darm1176jzf-s" > +makeoptions CONF_CFLAGS=3D"-mcpu=3Darm1176jzf-s" >=20 > files "../broadcom/bcm2835/files.bcm2835" >=20 >=20 > Modified: head/sys/conf/kern.mk > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- head/sys/conf/kern.mk Thu Jan 1 01:43:00 2015 = (r276487) > +++ head/sys/conf/kern.mk Thu Jan 1 02:00:04 2015 = (r276488) > @@ -164,4 +164,26 @@ CFLAGS+=3D -fstack-protector > CFLAGS+=3D -gdwarf-2 > .endif >=20 > +# A whole bunch of new default warnings in clang 3.5 subpress for now = until > +# this can be cleaned up. > +.if ${COMPILER_VERSION} >=3D 30500 > +CFLAGS.clang+=3D -Wno-pointer-sign -Wno-constant-conversion = -Wno-format \ > + -Wno-shift-count-negative -Wno-tautological-pointer-compare \ > + -Wno-shift-count-overflow -Wno-tautological-compare > +.endif > + > CFLAGS+=3D ${CFLAGS.${COMPILER_TYPE}} Note that for x86, all of these warnings were already either fixed or suppressed. I assume this will be fleshed out later? > +# Tell bmake not to mistake standard targets for things to be = searched for > +# or expect to ever be up-to-date. > +PHONY_NOTMAIN =3D afterdepend afterinstall all beforedepend = beforeinstall \ > + beforelinking build build-tools buildfiles buildincludes = \ > + checkdpadd clean cleandepend cleandir cleanobj configure = \ > + depend dependall distclean distribute exe \ > + html includes install installfiles installincludes lint = \ > + obj objlink objs objwarn realall realdepend \ > + realinstall regress subdir-all subdir-depend = subdir-install \ > + tags whereobj > + > +.PHONY: ${PHONY_NOTMAIN} > +.NOTMAIN: ${PHONY_NOTMAIN} >=20 Was this latter part intentional? It does not seem to have anything to do with clang at all. -Dimitry --Apple-Mail=_ED03ED01-C25D-4F80-8742-DB50BD9CB951 Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP using GPGMail -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.26 iEYEARECAAYFAlSkxHMACgkQsF6jCi4glqMcQQCfTzTjt8h1mut6rhtgKlLde3V9 sjoAn09YyhMD9USWNJFSYN8oPtzB04As =51DO -----END PGP SIGNATURE----- --Apple-Mail=_ED03ED01-C25D-4F80-8742-DB50BD9CB951--