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

[Michael Niedermayer]

On Wed, May 20, 2020 at 01:02:39PM -0700, Dale Curtis wrote:
> On Wed, May 20, 2020 at 1:17 AM Michael Niedermayer <michael at niedermayer.cc>
> wrote:
> 
> >
> > In file included from ./libavutil/avutil.h:296:0,
> >                  from ./libavutil/log.h:25,
> >                  from ./libavutil/thread.h:34,
> >                  from libavdevice/alldevices.c:22:
> > ./libavutil/common.h: In function ‘av_sat_add64_c’:
> > ./libavutil/common.h:302:105: error: missing binary operator before token
> > "("
> >  #if (!defined(__INTEL_COMPILER) && AV_GCC_VERSION_AT_LEAST(5,1)) ||
> > (defined(__clang__) && __has_builtin(__builtin_add_overflow))
> >
> >
> What compiler is this? This is a seriously ancient function, so surprised
> it's not present, but a && __has_builtin would fix this.

gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
used with ccache on a x86-64

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200521/8fb70de7/attachment.sig>