[FFmpeg-cvslog] checkasm: x86: post commit review fixes
Janne Grunau
git at videolan.org
Sat Jan 2 13:27:05 CET 2016
ffmpeg | branch: master | Janne Grunau <janne-libav at jannau.net> | Tue Dec 22 22:51:55 2015 +0100| [f0f54117c8f206e8045d301c2eb975b26e9f263d] | committer: Janne Grunau
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.
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=f0f54117c8f206e8045d301c2eb975b26e9f263d
---
tests/checkasm/x86/checkasm.asm | 20 +++++++++-----------
1 file changed, 9 insertions(+), 11 deletions(-)
diff --git a/tests/checkasm/x86/checkasm.asm b/tests/checkasm/x86/checkasm.asm
index 147d7a7..52d10ae 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
More information about the ffmpeg-cvslog
mailing list