Hi, 2016-04-27 13:44 GMT+02:00 Hendrik Leppkes <[email protected]>: >> +#if AV_GCC_VERSION_AT_LEAST(4,3) >> +# define av_unlikely(x) __builtin_expect((x) != 0, 0) >> +#else >> +# define av_unlikely(x) x >> +#endif >> + > > I dont like this functionality. Ever tried to read GNU sources where > this is used commonly? It gets practically unreadable.
It's not likely this measurably improves performance, but it is used in several projects, e.g. the linux kernel. I guess this should be used sparingly, and with benchmark reports to back the few cases. -- Christophe _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
