Re: [Qemu-devel] [PATCH] Update the avx2 configure test to be compatible with clang

2019-08-09 Thread Rebecca Cran
On 2019-08-08 23:22, Richard Henderson wrote: > > No, this means we're not testing what we need: > > We need to compile exactly one function using avx2. > > The other functions should be compiled with sse4 and sse2, respectively, and > we > choose between them by testing cpuid bits at startup. If

Re: [Qemu-devel] [PATCH] Update the avx2 configure test to be compatible with clang

2019-08-08 Thread Richard Henderson
On 8/8/19 9:19 PM, Rebecca Cran wrote: > clang doesn't support the GCC pragma to enable AVX2, but instead > requires the command line option -mavx2. Since GCC also supports that, > remove the pragma lines and add the -mavx2 option when building the > test. No, this means we're not testing what we

Re: [Qemu-devel] [PATCH] Update the avx2 configure test to be compatible with clang

2019-08-08 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190809041952.57302-1-rebe...@bsdio.com/ Hi, This series failed the asan build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #!/bin/bash make d

[Qemu-devel] [PATCH] Update the avx2 configure test to be compatible with clang

2019-08-08 Thread Rebecca Cran
clang doesn't support the GCC pragma to enable AVX2, but instead requires the command line option -mavx2. Since GCC also supports that, remove the pragma lines and add the -mavx2 option when building the test. Signed-off-by: Rebecca Cran --- configure | 4 +--- 1 file changed, 1 insertion(+), 3