Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Jun 2016 10:44:12 -0700
From:      Bryan Drewery <bdrewery@FreeBSD.org>
To:        Dimitry Andric <dim@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org, ache@FreeBSD.org
Subject:   Re: svn commit: r301277 - in head: . lib/clang lib/clang/include/clang/Basic lib/clang/libclangbasic
Message-ID:  <e2078a66-7365-aa88-e3ae-7fe6e02d386b@FreeBSD.org>
In-Reply-To: <201606031617.u53GHaEB088345@repo.freebsd.org>
References:  <201606031617.u53GHaEB088345@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--VNjnRLneTl6CdwTCjemOb7pB3xE25sjgf
Content-Type: multipart/mixed; boundary="0sMtmcN35gA48UnWcExJWjSo89AmUow80"
From: Bryan Drewery <bdrewery@FreeBSD.org>
To: Dimitry Andric <dim@FreeBSD.org>, src-committers@freebsd.org,
 svn-src-all@freebsd.org, svn-src-head@freebsd.org, ache@FreeBSD.org
Message-ID: <e2078a66-7365-aa88-e3ae-7fe6e02d386b@FreeBSD.org>
Subject: Re: svn commit: r301277 - in head: . lib/clang
 lib/clang/include/clang/Basic lib/clang/libclangbasic
References: <201606031617.u53GHaEB088345@repo.freebsd.org>
In-Reply-To: <201606031617.u53GHaEB088345@repo.freebsd.org>

--0sMtmcN35gA48UnWcExJWjSo89AmUow80
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

Thanks!

I think this helps Andrey's recent clang rebuild issues as well since
the quite-common file was being touched often.

On 6/3/2016 9:17 AM, Dimitry Andric wrote:
> Author: dim
> Date: Fri Jun  3 16:17:36 2016
> New Revision: 301277
> URL: https://svnweb.freebsd.org/changeset/base/301277
>=20
> Log:
>   For clang, move the definition of FREEBSD_CC_VERSION into its own hea=
der
>   file, lib/clang/freebsd_cc_version.h, instead of reusing Version.inc.=

>   The header is only included from one .cpp file in the clang tree.
>  =20
>   This minimizes the number of .cpp files that need to be rebuilt if th=
e
>   version is bumped.
>  =20
>   Discussed with:	bdrewery
>=20
> Added:
>   head/lib/clang/freebsd_cc_version.h   (contents, props changed)
> Modified:
>   head/Makefile.inc1
>   head/lib/clang/include/clang/Basic/Version.inc
>   head/lib/clang/libclangbasic/Makefile
>=20
> Modified: head/Makefile.inc1
> =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/Makefile.inc1	Fri Jun  3 15:33:21 2016	(r301276)
> +++ head/Makefile.inc1	Fri Jun  3 16:17:36 2016	(r301277)
> @@ -99,7 +99,7 @@ _expected_compiler_type=3D	gcc
>  .if ${_expected_compiler_type} =3D=3D "clang"
>  CROSS_COMPILER_FREEBSD_VERSION!=3D \
>  	awk '$$2 =3D=3D "FREEBSD_CC_VERSION" {printf("%d\n", $$3)}' \
> -	${SRCDIR}/lib/clang/include/clang/Basic/Version.inc || echo unknown
> +	${SRCDIR}/lib/clang/freebsd_cc_version.h || echo unknown
>  CROSS_COMPILER_VERSION!=3D \
>  	awk '$$2 =3D=3D "CLANG_VERSION" {split($$3, a, "."); print a[1] * 100=
00 + a[2] * 100 + a[3]}' \
>  	${SRCDIR}/lib/clang/include/clang/Basic/Version.inc || echo unknown
>=20
> Added: head/lib/clang/freebsd_cc_version.h
> =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
> --- /dev/null	00:00:00 1970	(empty, because file is newly added)
> +++ head/lib/clang/freebsd_cc_version.h	Fri Jun  3 16:17:36 2016	(r3012=
77)
> @@ -0,0 +1,3 @@
> +/* $FreeBSD$ */
> +
> +#define	FREEBSD_CC_VERSION		1100004
>=20
> Modified: head/lib/clang/include/clang/Basic/Version.inc
> =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/lib/clang/include/clang/Basic/Version.inc	Fri Jun  3 15:33:21 =
2016	(r301276)
> +++ head/lib/clang/include/clang/Basic/Version.inc	Fri Jun  3 16:17:36 =
2016	(r301277)
> @@ -8,5 +8,3 @@
>  #define	CLANG_VENDOR			"FreeBSD "
> =20
>  #define	SVN_REVISION			"262564"
> -
> -#define	FREEBSD_CC_VERSION		1100004U
>=20
> Modified: head/lib/clang/libclangbasic/Makefile
> =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/lib/clang/libclangbasic/Makefile	Fri Jun  3 15:33:21 2016	(r30=
1276)
> +++ head/lib/clang/libclangbasic/Makefile	Fri Jun  3 16:17:36 2016	(r30=
1277)
> @@ -50,3 +50,6 @@ TGHDRS=3D	AttrHasAttributeImpl \
> =20
>  # XX: work around GCC bug 67888
>  CFLAGS.gcc +=3D -fpermissive
> +
> +# Ensure FREEBSD_CC_VERSION is defined for Targets.cpp
> +CFLAGS.Targets.cpp+=3D -include ../freebsd_cc_version.h
>=20


--=20
Regards,
Bryan Drewery


--0sMtmcN35gA48UnWcExJWjSo89AmUow80--

--VNjnRLneTl6CdwTCjemOb7pB3xE25sjgf
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQEcBAEBAgAGBQJXUcHsAAoJEDXXcbtuRpfPDcgH/1CK5mooorpkCojxhK1ZYA+r
DJGB3wfaz58PijqXTNgLi8H/AwtN5MXLV9VDVYKfD14/Hl13qSg8aHSbclAtPohG
HP0ZTrOXmkrQ+pRNfn2AC/veGoszxohbg4D9gGVGWxayXXnKRrXRLEgZuxNvL8RU
hBUMhx7Y1mt38JbQ2J8tWQuG4HQCPfx9LDR8xVyETXixrwwEsFIC+dpcG9AJK/qf
EtWP3Qx+ZtkpNpk+Yd2Xs+V5wQ9Ya8E6pyvbvl/DdAVe6VHywb8EofFY0Hc5lOvV
qinTcoN4ELAFcljADqUafM4PPDJkbGpVEed31yfWTC11VzFjpdkzTtbN5scom08=
=HxT/
-----END PGP SIGNATURE-----

--VNjnRLneTl6CdwTCjemOb7pB3xE25sjgf--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?e2078a66-7365-aa88-e3ae-7fe6e02d386b>