On 19/02/13 00:32, Diego Biurrun wrote:
> The init functions marked as av_cold have to be executed in any case,
> so there is no gain from trying to mark paths leading to such functions
> as unlikely.
> ---
> 
> 2/2 sort of depends on this one, as the header shuffling made it necessary
> to #include attributes.h directly or, as I did here, to drop the av_cold
> attributes.

The rbelf shows some slight increase in size but the overall effect
should be negligible (those functions should be called by cold functions
themselves most if not all the time).

Patch fine for me (should speed up compiling a little and recompiling a
lot as positive tradeoff)

(GCC reports cold to do slightly different things, one part of the
documentation pretty much makes it similar to the
builtin_expect/unlikely the other describes it closer to call-once/seldom)

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to