diff options
| author | Martin Storsjö <martin@martin.st> | 2016-10-18 13:48:56 +0300 |
|---|---|---|
| committer | Martin Storsjö <martin@martin.st> | 2016-10-18 16:17:12 +0300 |
| commit | dd5d4a0e1e3a30a254d1a57ecbdcedf230c6014b (patch) | |
| tree | f46449808602b3c3bc7dd886e1c1a8c935f22812 /tests/checkasm | |
| parent | 7911186ed616ae81dd8617d6d0e8b08c818db9d8 (diff) | |
| download | ffmpeg-dd5d4a0e1e3a30a254d1a57ecbdcedf230c6014b.tar.gz | |
checkasm: aarch64: Don't clobber x29 in checkasm_stack_clobber
x29 (FP) is a callee saved register and should be restored on
return. Instead of backing up x29 and restoring it here, back up
sp in a register that we are allowed to overwrite.
This fixes crashes in checkasm on aarch64 since f1b3e1313851.
For some reason, gcc builds didn't crash, but clang builds do.
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'tests/checkasm')
| -rw-r--r-- | tests/checkasm/aarch64/checkasm.S | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/checkasm/aarch64/checkasm.S b/tests/checkasm/aarch64/checkasm.S index 65b352104e..bc5ed9ea09 100644 --- a/tests/checkasm/aarch64/checkasm.S +++ b/tests/checkasm/aarch64/checkasm.S @@ -55,13 +55,13 @@ endconst #define CLOBBER_STACK ((8*MAX_ARGS + 15) & ~15) function checkasm_stack_clobber, export=1 - mov x29, sp + mov x3, sp mov x2, #CLOBBER_STACK 1: stp x0, x1, [sp, #-16]! subs x2, x2, #16 b.gt 1b - mov sp, x29 + mov sp, x3 ret endfunc |