Bump. I have 5 integer overflow fuzzing issues awaiting our resolution of this discussion. Thanks.
- dale On Fri, May 1, 2020 at 2:53 PM Dale Curtis <[email protected]> wrote: > 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".
