[FFmpeg-devel] [PATCH v6 1/4] lavc: add FLIF decoding support

Nicolas George george at nsup.org
Thu Aug 27 21:26:29 EEST 2020


Lynne (12020-08-27):
> We always inline bitstream reading functions, and this one is no different.
> Its really necessary for speed, and well worth the additional binary size.
> There's no memory cache gains to be had here, not with the huge
> several hundred instruction lookaheads modern CPUs have had for decades,
> just function call overhead and stack spillage to be saved on.
> If we could only put av_always_inline on a single type of function in decoders,
> it would have to be in bitstream reading, so please use that here.

The functions I asked not to inline were huge and calling each other,
growing exponentially, not the bitstream-reading ones.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200827/75818587/attachment.sig>


More information about the ffmpeg-devel mailing list