From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49473) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gS5DD-0005TD-EL for qemu-devel@nongnu.org; Wed, 28 Nov 2018 14:09:28 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gS5DC-0001Xy-6t for qemu-devel@nongnu.org; Wed, 28 Nov 2018 14:09:27 -0500 References: <1543399094-2260-1-git-send-email-thuth@redhat.com> <20181128160525.GP24355@redhat.com> <3d95da16-5f2a-2ec8-a674-189b44eab222@redhat.com> <20181128165803.GT24355@redhat.com> From: Thomas Huth Message-ID: <7b33a961-b37b-f8ce-a77f-5c1bbd19ac42@redhat.com> Date: Wed, 28 Nov 2018 20:09:03 +0100 MIME-Version: 1.0 In-Reply-To: <20181128165803.GT24355@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH for-3.1?] target/arm/sve_helper: Fix compilation with clang 3.4 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "=?UTF-8?Q?Daniel_P._Berrang=c3=a9?=" Cc: Peter Maydell , qemu-devel@nongnu.org, qemu-arm@nongnu.org, Richard Henderson On 2018-11-28 17:58, Daniel P. Berrang=C3=A9 wrote: > On Wed, Nov 28, 2018 at 05:50:38PM +0100, Thomas Huth wrote: >> On 2018-11-28 17:05, Daniel P. Berrang=C3=A9 wrote: >>> On Wed, Nov 28, 2018 at 10:58:14AM +0100, Thomas Huth wrote: >>>> Clang 3.4 does not know the "flatten" attribute yet. We've already >>>> introduced the QEMU_FLATTEN macro for this in commit 97ff87c0ed020c2= , >>>> so use this macro now here, too, to fix this issue. >>>> >>>> Signed-off-by: Thomas Huth >>> >>> Although we currently only aim to support CLang and GCC, we don't hav= e >>> any checks for a minimum version of these two tools in configure. >>> >>> So I was wondering if we actually still need to support CLang 3.4 at = all >>> in QEMU, given our targetted build platforms. >>> >>> Unfortunately repology.org is failing me in reporting clang versions >>> for most distros, so I don't have an answer yet. I do see that Debian >>> Jessie however ships clang =3D=3D 3.5 and that's usually the oldest d= istro >>> if RHEL hasn't got something older. >> >> I'm using RHEL7 and the clang that I get "for free" is 3.4.2 via EPEL. >=20 > Hmm, raises interesting question of whether we should consider 3rd > party repos for purposes of determining our min versions. >=20 > I'd pretty much intended to only care about vendor provided repos, > in which case RHEL7 would be considered to lack clang, for purposes > of deciding min versions. >=20 > EPEL is a bit murky because while it is a 3rd party repo, many users > of RHEL consider to be a defacto standard repo. Looks like there's also the possibility to get newer versions of Clang on top of RHEL7: https://developers.redhat.com/blog/2018/07/07/yum-install-gcc7-clang/ ... so fine for me if we use another version as the minimum. But if we decide to use 3.4 as minimum version, I'll keep my current compiler to be able to report problems... Thomas