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

Michael Niedermayer michael at niedermayer.cc
Wed May 27 23:16:22 EEST 2020


On Wed, May 27, 2020 at 11:27:14AM -0700, Dale Curtis wrote:
> On Wed, May 27, 2020 at 11:10 AM Michael Niedermayer <michael at niedermayer.cc>
> wrote:
> 
> > so the builtin does better and both locally do better than clang on the web
> > tool
> >
> 
> Ah, sorry I forgot godbolt is compiling without any options, if you type
> -O9 in the compiler options fields you get:
> clang: https://godbolt.org/z/CAMXer
> gcc:  https://godbolt.org/z/EKqGt6
> 
> Which is similar to what you've gotten above; with the builtin having the
> smallest code.

good, i think this has been tested quite well now ...
will apply

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200527/09a258ef/attachment.sig>


More information about the ffmpeg-devel mailing list