Kostya Shishkov <[email protected]> writes: > On Sat, Aug 04, 2012 at 03:44:35AM +0100, Mans Rullgard wrote: >> The main benefit of inlining this function is from constant >> propagation for the 'field_based' argument. Instead of inlining >> all calls, create two versions of the fuction for field_based >> values of 0 and 1. >> >> Signed-off-by: Mans Rullgard <[email protected]> >> --- >> libavcodec/mpegvideo_motion.c | 67 >> +++++++++++++++++++++++++++---------------- >> 1 file changed, 42 insertions(+), 25 deletions(-) > > "Compiler does what it likes to (often randomly), not what you mean it to do"
GCC can, in theory, do this kind of function cloning. In practice, it rarely does quite what you want. -- Måns Rullgård [email protected] _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
