[FFmpeg-devel] [PATCH] checkasm/aarch64: fix tests returning a float
Matthieu Bouron
matthieu.bouron at gmail.com
Wed Jun 21 22:46:20 EEST 2017
On Mon, Jun 19, 2017 at 11:08:39AM +0200, Matthieu Bouron wrote:
> Avoids overriding v0 (which containins the result of the tested
> function) in checkasm_call_checked.
>
> Also properly calls checkasm_call_checked.
> ---
> tests/checkasm/aarch64/checkasm.S | 8 ++++----
> tests/checkasm/checkasm.h | 2 ++
> 2 files changed, 6 insertions(+), 4 deletions(-)
>
> diff --git a/tests/checkasm/aarch64/checkasm.S b/tests/checkasm/aarch64/checkasm.S
> index 53a2a478dc..75a9a56143 100644
> --- a/tests/checkasm/aarch64/checkasm.S
> +++ b/tests/checkasm/aarch64/checkasm.S
> @@ -112,10 +112,10 @@ function checkasm_checked_call, export=1
> movi v3.8h, #0
>
> .macro check_reg_neon reg1, reg2
> - ldr q0, [x9], #16
> - uzp1 v1.2d, v\reg1\().2d, v\reg2\().2d
> - eor v0.16b, v0.16b, v1.16b
> - orr v3.16b, v3.16b, v0.16b
> + ldr q1, [x9], #16
> + uzp1 v2.2d, v\reg1\().2d, v\reg2\().2d
> + eor v1.16b, v1.16b, v2.16b
> + orr v3.16b, v3.16b, v1.16b
> .endm
> check_reg_neon 8, 9
> check_reg_neon 10, 11
> diff --git a/tests/checkasm/checkasm.h b/tests/checkasm/checkasm.h
> index 695d871220..5249669fba 100644
> --- a/tests/checkasm/checkasm.h
> +++ b/tests/checkasm/checkasm.h
> @@ -145,6 +145,8 @@ void checkasm_stack_clobber(uint64_t clobber, ...);
> void checkasm_checked_call(void *func, ...);
> #define declare_new(ret, ...) ret (*checked_call)(void *, int, int, int, int, int, int, int, __VA_ARGS__)\
> = (void *)checkasm_checked_call;
> +#define declare_new_float(ret, ...) ret (*checked_call)(void *, int, int, int, int, int, int, int, __VA_ARGS__)\
> + = (void *)checkasm_checked_call;
> #define CLOB (UINT64_C(0xdeadbeefdeadbeef))
> #define call_new(...) (checkasm_stack_clobber(CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,\
> CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB,CLOB),\
> --
> 2.13.1
>
Ping.
--
Matthieu B.
More information about the ffmpeg-devel
mailing list