[FFmpeg-devel] [PATCH 1/4] swscale/x86/rgb2rgb_template: Remove unnecessary SFENCE

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat Jun 8 22:44:37 EEST 2024


Andreas Rheinhardt:
> The ff_nv12ToUV_* functions don't use non-temporal stores
> at all.
> 
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libswscale/x86/rgb2rgb_template.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/libswscale/x86/rgb2rgb_template.c b/libswscale/x86/rgb2rgb_template.c
> index edbacea784..e4e884827c 100644
> --- a/libswscale/x86/rgb2rgb_template.c
> +++ b/libswscale/x86/rgb2rgb_template.c
> @@ -1837,10 +1837,6 @@ static void RENAME(deinterleaveBytes)(const uint8_t *src, uint8_t *dst1, uint8_t
>          dst1 += dst1Stride;
>          dst2 += dst2Stride;
>      }
> -    __asm__(
> -            SFENCE"     \n\t"
> -            ::: "memory"
> -            );
>  }
>  #endif /* COMPILE_TEMPLATE_SSE2 && HAVE_X86ASM */
>  #endif /* !COMPILE_TEMPLATE_AVX || HAVE_AVX_EXTERNAL */

Will apply this patchset tomorrow unless there are objections.

- Andreas



More information about the ffmpeg-devel mailing list