From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:32775) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YnBF9-0005hR-A2 for qemu-devel@nongnu.org; Tue, 28 Apr 2015 15:32:32 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YnBF5-00045B-7C for qemu-devel@nongnu.org; Tue, 28 Apr 2015 15:32:31 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40225) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YnBF4-000454-Vl for qemu-devel@nongnu.org; Tue, 28 Apr 2015 15:32:27 -0400 Message-ID: <553FE045.3000106@redhat.com> Date: Tue, 28 Apr 2015 15:32:21 -0400 From: John Snow MIME-Version: 1.0 References: <1427324259-1481-1-git-send-email-jsnow@redhat.com> <20150330151051.GO25181@stefanha-thinkpad.redhat.com> In-Reply-To: <20150330151051.GO25181@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v5 0/4] configure: clang 3.5.0 build fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: peter.maydell@linaro.org, qemu-devel@nongnu.org, stefanha@redhat.com On 03/30/2015 11:10 AM, Stefan Hajnoczi wrote: > On Wed, Mar 25, 2015 at 06:57:35PM -0400, John Snow wrote: >> QEMU does not compile cleanly under clang 3.5.0. These patches eliminate the >> avalanche of warnings and make the build usable. >> >> The result is that you *should* be able to use clang 3.5.0 *with* ccache and >> -Werror and produce all targets. >> >> Regrettably, this is v5 because I overlooked deleting "--keep-tmpc" in the >> previous version. Sorry mom, I let the homestead down. We'll be eating dirt >> this winter. I can always go to the city for work, please don't cry. >> >> v4: >> - Enable ccache workaround for configurations without -Werror, >> to suppress warnings as well. >> - Removed optimization from cc_has_warning_flag() >> - Renamed ccache variable to ccache_cpp2 >> >> John Snow (3): >> configure: factor out supported flag check >> configure: silence glib unknown attribute __alloc_size__ >> configure: Add workaround for ccache and clang >> >> Stefan Hajnoczi (1): >> configure: handle clang -nopie argument warning >> >> configure | 81 ++++++++++++++++++++++++++++++++++++++++++++++++++------------- >> 1 file changed, 65 insertions(+), 16 deletions(-) > > Reviewed-by: Stefan Hajnoczi > ping for 3.4