From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:57985) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zruk7-0001V3-3P for qemu-devel@nongnu.org; Thu, 29 Oct 2015 17:28:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zruk2-0005wG-W1 for qemu-devel@nongnu.org; Thu, 29 Oct 2015 17:28:19 -0400 Received: from mx1.redhat.com ([209.132.183.28]:48954) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zruk2-0005vU-Pu for qemu-devel@nongnu.org; Thu, 29 Oct 2015 17:28:14 -0400 References: <1446150168-9632-1-git-send-email-jsnow@redhat.com> <563281A8.60604@redhat.com> <56328DB2.5010205@redhat.com> From: John Snow Message-ID: <56328F6D.2040303@redhat.com> Date: Thu, 29 Oct 2015 17:28:13 -0400 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] configure: workaround for Clang 3.5.0 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , Laszlo Ersek Cc: Paolo Bonzini , QEMU Developers On 10/29/2015 05:25 PM, Peter Maydell wrote: > On 29 October 2015 at 21:20, Laszlo Ersek wrote: >> On 10/29/15 21:29, John Snow wrote: >>> Here's a good takeaway quote: >>> >>> 'Also, _FORTIFY_SOURCE + glibc + clang is not supported and does not >>> work (for instance, it relies on __builtin_va_pack_len and friends, >>> which we have no intention of supporting), so glibc compatibility is >>> unlikely to be a strong motivator for a change here.' >> >> this quote would be compelling enough for me to disable _FORTIFY_SOURCE >> when clang is seen, no questions asked. The above is a public no-support >> statement from an apparently core clang developer, so "it happens to >> build without errors with version X.Y.Z." just don't cut it. A positive >> claim (bugzilla comment, release note, etc) would be necessary. >> >> ... As far as I'm concerned, of course. :) > > I think I would agree with that. > > thanks > -- PMM > OK. I'll spin a new one that disables FORTIFY SOURCE without test if it detects you are using Clang. (I am curious as to why it now magically works in later versions of Clang though, despite such a strong "no support" statement from the developer above. I might do a little digging.) --js