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

[Dale Curtis]

On Fri, May 1, 2020 at 12:50 PM Dale Curtis <dalecurtis at chromium.org> 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.
>
>
Whoops, I misread the release notes. It's 6.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66970 I guess configure is the
only way unless we limit to GCC6+.

- dale