]> xenbits.xensource.com Git - people/hx242/xen.git/commitdiff
x86emul: correct test harness {evex} assembler capability check
authorJan Beulich <jbeulich@suse.com>
Fri, 22 May 2020 12:35:04 +0000 (14:35 +0200)
committerJan Beulich <jbeulich@suse.com>
Fri, 22 May 2020 12:35:04 +0000 (14:35 +0200)
The {evex} pseudo prefix gets rejected by gas for insns not allowing
EVEX encoding. Except there's a gas bug due to which its check gets
bypassed for insns without operands. Let's not rely on that bug to
remain there.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
tools/tests/x86_emulator/Makefile

index 7ce882cb78881f71bb139c64a23c66801058c75b..b414773810d78b93511444b57103ac425cfb6998 100644 (file)
@@ -112,7 +112,7 @@ $(foreach flavor,$(SIMD) $(FMA),$(eval $(call simd-check-cc,$(flavor))))
 
 # Also explicitly check for {evex} pseudo-prefix support, which got introduced
 # only after AVX512F and some of its extensions.
-TARGET-$(shell echo 'asm("{evex} vzeroall");' | $(CC) -x c -c -o /dev/null - || echo y) :=
+TARGET-$(shell echo 'asm("{evex} vmovaps %xmm0$(comma)%xmm0");' | $(CC) -x c -c -o /dev/null - || echo y) :=
 
 ifeq ($(TARGET-y),)
 $(warning Test harness not built, use newer compiler than "$(CC)" (version $(shell $(CC) -dumpversion)) and an "{evex}" capable assembler)