diff options
author | Janne Grunau <janne-libav@jannau.net> | 2015-12-23 00:51:55 +0300 |
---|---|---|
committer | Janne Grunau <janne-libav@jannau.net> | 2015-12-29 14:50:38 +0300 |
commit | f0f54117c8f206e8045d301c2eb975b26e9f263d (patch) | |
tree | d917b67ae718961b2bf1996d62b5a929c979b7a0 /tests | |
parent | f4f27e4cf1013c55b2c7df359ce8d58ee922662c (diff) |
checkasm: x86: post commit review fixes
Check the full FPU tag word instead of only the lower half and simplify
the comparison.
Use upper-case function base name as macro name to instantiate both
checked_call variants.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/checkasm/x86/checkasm.asm | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/tests/checkasm/x86/checkasm.asm b/tests/checkasm/x86/checkasm.asm index 147d7a7497..52d10aec5c 100644 --- a/tests/checkasm/x86/checkasm.asm +++ b/tests/checkasm/x86/checkasm.asm @@ -98,7 +98,7 @@ cglobal stack_clobber, 1,2 ; void checkasm_checked_call(void *func, ...) ;----------------------------------------------------------------------------- INIT_XMM -%macro check_call 0-1 +%macro CHECKED_CALL 0-1 cglobal checked_call%1, 2,15,16,max_args*8+8 mov t0, r0 @@ -171,9 +171,8 @@ cglobal checked_call%1, 2,15,16,max_args*8+8 .clobber_ok: %ifnid %1, _emms fstenv [rsp] - mov r9h, [rsp + 8] - add r9h, 1 - jz .emms_ok + cmp word [rsp + 8], 0xffff + je .emms_ok report_fail error_message_emms emms .emms_ok: @@ -201,7 +200,7 @@ cglobal checked_call%1, 2,15,16,max_args*8+8 mov eax, r3 %endmacro -%macro check_call 0-1 +%macro CHECKED_CALL 0-1 ;----------------------------------------------------------------------------- ; void checkasm_checked_call(void *func, ...) ;----------------------------------------------------------------------------- @@ -225,10 +224,9 @@ cglobal checked_call%1, 1,7 report_fail error_message .clobber_ok: %ifnid %1, _emms - fstenv [rsp] - mov r3h, [rsp + 8] - add r3h, 1 - jz .emms_ok + fstenv [esp] + cmp word [esp + 8], 0xffff + je .emms_ok report_fail error_message_emms emms .emms_ok: @@ -241,5 +239,5 @@ cglobal checked_call%1, 1,7 %endif ; ARCH_X86_64 -check_call -check_call _emms +CHECKED_CALL +CHECKED_CALL _emms |