Re: [Qemu-devel] [PATCH v2 0/2] AVX2 configure fixes

[Li, Liang Z]

> Cc: amit.s...@redhat.com
> Subject: [PATCH v2 0/2] AVX2 configure fixes
> 
> 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
> 
> 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 | 15 +++
>  util/cutils.c |  8 +---
>  2 files changed, 12 insertions(+), 11 deletions(-)
> 
> --
> 2.7.4

Looks good. Thank!

Reviewed-by: Liang Li 

Liang

[Qemu-devel] [PATCH v2 0/2] AVX2 configure fixes

[Dr. David Alan Gilbert (git)]

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

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 | 15 +++
 util/cutils.c |  8 +---
 2 files changed, 12 insertions(+), 11 deletions(-)

-- 
2.7.4