diff mbox series

x86emul: fix SIMD test overriding of VBROADCASTS{S,D}

Message ID 0519c820-f1ee-1b37-f1be-90bd1daf37ed@suse.com (mailing list archive)
State New, archived
Headers show
Series x86emul: fix SIMD test overriding of VBROADCASTS{S,D} | expand

Commit Message

Jan Beulich Feb. 11, 2022, 11:11 a.m. UTC 
Despite their suffixes these aren't scalar instructions, and hence the
128- and 256-bit EVEX forms may not be used without AVX512VL. Gcc11 ends
up generating such instances for simd-sg.c.

Signed-off-by: Jan Beulich <jbeulich@suse.com>

Comments

Andrew Cooper Feb. 11, 2022, 11:38 a.m. UTC | #1 
On 11/02/2022 11:11, Jan Beulich wrote:
> Despite their suffixes these aren't scalar instructions, and hence the
> 128- and 256-bit EVEX forms may not be used without AVX512VL. Gcc11 ends
> up generating such instances for simd-sg.c.
>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
diff mbox series

Patch

--- a/tools/tests/x86_emulator/simd.h
+++ b/tools/tests/x86_emulator/simd.h
@@ -250,7 +250,9 @@  asm ( ".macro override insn    \n\t"
 # define OVR_INT(n) OVR_BW(n); OVR_DQ(n)
 
 OVR_INT(broadcast);
+# ifdef __AVX512VL__
 OVR_SFP(broadcast);
+# endif
 OVR_SFP(comi);
 OVR_VFP(cvtdq2);
 OVR_INT(abs);