[FFmpeg-devel] [PATCH 2/2] Use gcc/clang builtins for av_sat_(add|sub)_64_c if available.

Carl Eugen Hoyos ceffmpeg at gmail.com
Fri May 1 21:07:05 EEST 2020


Am Fr., 1. Mai 2020 um 19:24 Uhr schrieb Dale Curtis <dalecurtis at chromium.org>:
>
> Signed-off-by: Dale Curtis <dalecurtis at chromium.org>
> ---
>  libavutil/common.h | 10 ++++++++++
>  1 file changed, 10 insertions(+)

Imo, this is guaranteed to break x86 compilation with some unusual
toolchain.
I looked carefully at all occurrences of AV_GCC_VERSION_AT_LEAST()
and I believe they are in fact different (not for x86 or combined with other
checks).

If you believe that the request for a configure check is unreasonable,
please commit.

Carl Eugen


More information about the ffmpeg-devel mailing list