On Fri, May 1, 2020 at 2:00 PM Carl Eugen Hoyos <[email protected]> wrote:
> Am Fr., 1. Mai 2020 um 22:16 Uhr schrieb Dale Curtis < > [email protected]>: > > > > On Fri, May 1, 2020 at 1:12 PM Carl Eugen Hoyos <[email protected]> > wrote: > > > > > Am Fr., 1. Mai 2020 um 22:06 Uhr schrieb James Almer < > [email protected]>: > > > > Just make the check > > > > > > > > (AV_GCC_VERSION_AT_LEAST(5,1) || defined(__clang__)) && > > > > !defined(__INTEL_COMPILER) > > > > > > And switch the conditions. > > > > Thanks. Done. > > Is there a reason why this doesn't use > __has_builtin(__builtin_add_overflow) > for clang? > Yes, prior to clang 10 it didn't work properly: https://clang.llvm.org/docs/LanguageExtensions.html#has-builtin - dale _______________________________________________ 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".
