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

[Carl Eugen Hoyos]

Am Fr., 1. Mai 2020 um 22:06 Uhr schrieb James Almer <jamrial at gmail.com>:
>
> On 5/1/2020 4:50 PM, Dale Curtis wrote:
> > On Fri, May 1, 2020 at 12:37 PM Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> >
> >>>
> >>> So ICC on Linux defines __GNUC__ >= 5 yet doesn't support these builtins?
> >>
> >> No, but yes, this is the effect.
> >>
> >
> > Does this patch work instead on the ICC compiler? GCC 4.2+ have support for
> > __has_builtin() which shouldn't be masqueraded by the ICC.
>
> Just make the check
>
> (AV_GCC_VERSION_AT_LEAST(5,1) || defined(__clang__)) &&
> !defined(__INTEL_COMPILER)

And switch the conditions.

Carl Eugen