From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45762) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bCOD7-0004Uz-NT for qemu-devel@nongnu.org; Mon, 13 Jun 2016 05:31:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bCOD3-0008Kp-CD for qemu-devel@nongnu.org; Mon, 13 Jun 2016 05:31:08 -0400 Received: from mx1.redhat.com ([209.132.183.28]:41797) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bCOD3-0008Kj-6e for qemu-devel@nongnu.org; Mon, 13 Jun 2016 05:31:05 -0400 References: <1465557378-24105-1-git-send-email-dgilbert@redhat.com> From: Paolo Bonzini Message-ID: <3a9bb75c-fe94-a0e5-051c-ce9b0b21d3e5@redhat.com> Date: Mon, 13 Jun 2016 11:31:00 +0200 MIME-Version: 1.0 In-Reply-To: <1465557378-24105-1-git-send-email-dgilbert@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 0/2] AVX2 configure fixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Dr. David Alan Gilbert (git)" , qemu-devel@nongnu.org, peter.maydell@linaro.org Cc: liang.z.li@intel.com, amit.shah@redhat.com, quintela@redhat.com On 10/06/2016 13:16, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Hi, > This pair of patches fixes a couple of issues that we found during test. > The first is that the configure test is pessimistic when compiled with -O2, > the second is that the explicit 4.9 gcc test is a bit coarse; I've removed > that test but beefed up the ./configure test to actually use the avx2 > intrinsics and that fails in the same way as the main code, so it works > in the same way as the explicit check but allows older gcc's to work > when -save-temps isn't used. > > Dave > > v3 > Use compile_object rather than compile_prog as suggested by Peter to > hopefully fail on his ccache setup. Queued, thanks. Paolo > v2 > Remove the explicit version check > Split the patches > > > Dr. David Alan Gilbert (2): > Make avx2 configure test work with -O2 > avx2 configure: Use primitives in test > > configure | 17 ++++++++++++----- > util/cutils.c | 8 +------- > 2 files changed, 13 insertions(+), 12 deletions(-) >