[FFmpeg-devel] [PATCH] change some | to + in h264_cabac

Alexander Strange astrange
Fri Mar 26 06:04:24 CET 2010


On Mar 25, 2010, at 8:09 AM, Michael Niedermayer wrote:

> On Thu, Mar 25, 2010 at 03:35:08AM -0400, Alexander Strange wrote:
>> 6 insns less on x86-64, since gcc doesn't know the range of get_cabac_noinline's return value and can't change or to lea (well, I'm not sure it can do that in the first place).
> 
>> No speed change.
> 
> :(
> 
> 
>> h264_cabac.c |   16 ++++++++--------
>> 1 file changed, 8 insertions(+), 8 deletions(-)
>> 9ec22d9c32e88dc95c4b6c241a7793e8925c45b3  0004-cabac-Use-instead-of-in-some-places.patch
>> From bf53d404b2ca048450177689cb3d418f286f5ce4 Mon Sep 17 00:00:00 2001
>> From: Alexander Strange <astrange at ithinksw.com>
>> Date: Wed, 24 Mar 2010 16:28:42 -0400
>> Subject: [PATCH 4/4] cabac: Use + instead of | in some places.
>> 
>> 6 insns less on x86-64.
> 
> ok

Applied.



More information about the ffmpeg-devel mailing list