On Wed, 26 Jul 2023, [email protected] wrote:

From: Reimar Döffinger <[email protected]>

ret can be given an argument instead.
This is also consistent with how other assembler code
in FFmpeg does it.
---
libavcodec/aarch64/hevcdsp_idct_neon.S | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/libavcodec/aarch64/hevcdsp_idct_neon.S 
b/libavcodec/aarch64/hevcdsp_idct_neon.S
index b7f23386a4..f7142c939c 100644
--- a/libavcodec/aarch64/hevcdsp_idct_neon.S
+++ b/libavcodec/aarch64/hevcdsp_idct_neon.S
@@ -617,8 +617,7 @@ function ff_hevc_idct_16x16_\bitdepth\()_neon, export=1

        add              sp,  sp,  #640

-        mov             x30, x15
-        ret
+        ret             x15
endfunc
.endm

@@ -814,8 +813,7 @@ function ff_hevc_idct_32x32_\bitdepth\()_neon, export=1
.endr

        add             sp,  sp,  #2432
-        mov             x30, x15
-        ret
+        ret             x15
endfunc
.endm

LGTM, assuming checkasm still passes.

// Martin
_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to