[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