From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43077) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b8fOF-00024g-9S for qemu-devel@nongnu.org; Thu, 02 Jun 2016 23:03:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1b8fOB-0005sQ-3w for qemu-devel@nongnu.org; Thu, 02 Jun 2016 23:03:14 -0400 Received: from mga03.intel.com ([134.134.136.65]:20840) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1b8fOA-0005rO-Rk for qemu-devel@nongnu.org; Thu, 02 Jun 2016 23:03:11 -0400 From: "Li, Liang Z" Date: Fri, 3 Jun 2016 03:02:43 +0000 Message-ID: References: <1464792879-30954-1-git-send-email-dgilbert@redhat.com> In-Reply-To: <1464792879-30954-1-git-send-email-dgilbert@redhat.com> Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH v2 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" , "pbonzini@redhat.com" , "peter.maydell@linaro.org" Cc: "amit.shah@redhat.com" > Cc: amit.shah@redhat.com > Subject: [PATCH v2 0/2] AVX2 configure fixes >=20 > From: "Dr. David Alan Gilbert" >=20 > 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 -O= 2, the > second is that the explicit 4.9 gcc test is a bit coarse; I've removed th= at test > but beefed up the ./configure test to actually use the avx2 intrinsics an= d that > fails in the same way as the main code, so it works in the same way as th= e > explicit check but allows older gcc's to work when -save-temps isn't used= . >=20 > Dave >=20 > v2 > Remove the explicit version check > Split the patches >=20 > Dr. David Alan Gilbert (2): > Make avx2 configure test work with -O2 > avx2 configure: Use primitives in test >=20 > configure | 15 +++++++++++---- > util/cutils.c | 8 +------- > 2 files changed, 12 insertions(+), 11 deletions(-) >=20 > -- > 2.7.4 Looks good. Thank! Reviewed-by: Liang Li Liang