On 08/08/14 22:38, Diego Biurrun wrote: > On Mon, Jul 21, 2014 at 02:21:09PM -0700, Diego Biurrun wrote: >> Inline assembly has no place in the general C code and the utility >> of this microoptimization is doubtful. >> --- >> >> This is a troll patch, I did not bother to benchmark.
Since you are on linux you should use perf, it makes such stuff non-issues. (as in, DO benchmark even silly stuff). > I had another look at this, the function is only used in other functions > that are part of init code and in vc1_decode_frame under > 'if (!context_initialized)', i.e. only once. > > This is a silly microoptimization and thus not worth benchmarking IMO... context_initalized is one of those strange stuff from mpegvideo... lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
