From: Jan Beulich Date: Tue, 18 Dec 2018 13:26:44 +0000 (+0100) Subject: x86emul/test: drop another instance of .byte X-Git-Tag: archive/raspbian/4.14.0+80-gd101b417b7-1+rpi1^2~63^2~2746 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=95d99966713da79c5cc83748767b2cb3d89cb127;p=xen.git x86emul/test: drop another instance of .byte Now that we require use of the {evex} pseudo-prefix, we can also use the q-suffixed encoding of VPCMPESTRI, which is available as of 2.29 just like {evex} is. Signed-off-by: Jan Beulich Acked-by: Andrew Cooper --- diff --git a/tools/tests/x86_emulator/test_x86_emulator.c b/tools/tests/x86_emulator/test_x86_emulator.c index 165c3c72b2..2806ddcb0f 100644 --- a/tools/tests/x86_emulator/test_x86_emulator.c +++ b/tools/tests/x86_emulator/test_x86_emulator.c @@ -3322,22 +3322,15 @@ int main(int argc, char **argv) { decl_insn(vpcmpestri); -#ifdef __x86_64__ - /* - * gas up to at least 2.27 doesn't honor explict "rex.w" for - * VEX/EVEX encoded instructions, and also doesn't provide any - * other means to control VEX.W. - */ asm volatile ( "movq %0, %%xmm2\n" +#ifdef __x86_64__ put_insn(vpcmpestri, - ".byte 0xC4, 0xE3, 0xF9, 0x61, 0x16, 0x7A") - :: "m" (res[0]) ); + "vpcmpestriq $0b01111010, (%1), %%xmm2") #else - asm volatile ( "movq %0, %%xmm2\n" put_insn(vpcmpestri, "vpcmpestri $0b01111010, (%1), %%xmm2") - :: "m" (res[0]), "S" (NULL) ); #endif + :: "m" (res[0]), "S" (NULL) ); set_insn(vpcmpestri); #ifdef __x86_64__