From owner-svn-ports-all@freebsd.org Thu Dec 28 07:14:01 2017 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 4E122EABCB1; Thu, 28 Dec 2017 07:14:01 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: from freefall.freebsd.org (freefall.freebsd.org [96.47.72.132]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "freefall.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 2DC606F321; Thu, 28 Dec 2017 07:14:01 +0000 (UTC) (envelope-from danfe@freebsd.org) Received: by freefall.freebsd.org (Postfix, from userid 1033) id 687E544A2; Thu, 28 Dec 2017 07:14:00 +0000 (UTC) Date: Thu, 28 Dec 2017 07:14:00 +0000 From: Alexey Dokuchaev To: Yuri Victorovich Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r457438 - head/devel/ispc Message-ID: <20171228071400.GA42910@FreeBSD.org> References: <201712280506.vBS56ZSj011894@repo.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201712280506.vBS56ZSj011894@repo.freebsd.org> User-Agent: Mutt/1.9.2 (2017-12-15) X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Dec 2017 07:14:01 -0000 On Thu, Dec 28, 2017 at 05:06:35AM +0000, Yuri Victorovich wrote: > New Revision: 457438 > URL: https://svnweb.freebsd.org/changeset/ports/457438 > > Log: > devel/ispc: Changed the dependency to clang40 > > Reported by: danfe (asked to change the dependency of ISPC > for bug#224066 for graphics/embree update) > Differential Revision: https://reviews.freebsd.org/D13612 > > Modified: head/devel/ispc/Makefile > ... > @@ -15,14 +16,14 @@ LICENSE_FILE= ${WRKSRC}/LICENSE.txt > > ALL_TARGET= default > MAKE_ENV= LLVM_HOME="${LOCALBASE}" > -CLANG_VERSION= 38 > +CLANG_VERSION= 40 Thanks. But, now we have another problem... Originally, I was a bit reluctant towards making ISPC enabled by default in `graphics/embree' because ISPC wanted Clang/LLVM version 3.8 while our X.org/Mesa stack was using 4.0. But just couple of days ago X.org/Mesa had switched to 5.0, and I'm not sure what to do about this yet. I'm currently building LLVM 5.0 to see how much is it worse compared to 4.0. If it's not that bad, we might simply follow X.org/Mesa and set CLANG_VERSION to 50. Or we could implement LLVM version logic in bsd.default-versions.mk and teach ISPC to use it so it stays in sync with whatever X.org/Mesa wants regardless of the particular version. ./danfe