[FFmpeg-devel] [PATCH] simplify decode_cabac_residual specializations

[Alexander Strange]

On Mar 25, 2010, at 6:33 PM, Michael Niedermayer wrote:

> On Thu, Mar 25, 2010 at 02:44:19AM -0400, Alexander Strange wrote:
>> I can't remember why I didn't do this in the first place...
>> 
>> Slightly faster with gcc 4.2, only due to random inlining changes.
>> 
> 
>> h264_cabac.c |   23 ++++++-----------------
>> 1 file changed, 6 insertions(+), 17 deletions(-)
>> 8624c304348f81a7a503d4dedd106dc9d2f1d698  0001-h264-Simplify-decode_cabac_residual-specialization.patch
>> From d08dfcb07f73787fad4938865056ffeea61922cc Mon Sep 17 00:00:00 2001
>> From: Alexander Strange <astrange at ithinksw.com>
>> Date: Wed, 17 Mar 2010 05:24:16 -0400
>> Subject: [PATCH 1/4] h264: Simplify decode_cabac_residual() specialization.
>> 
>> ---
>> libavcodec/h264_cabac.c |   23 ++++++-----------------
>> 1 files changed, 6 insertions(+), 17 deletions(-)
> 
> ok

Applied (and found that it improved inlining with clang and the arm-apple-darwin gcc).

> [...]
>> h264_cabac.c |    8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>> af5d0c541fe0772696df4cc64cf92404e637393c  0002-h264-Remove-unused-function-argument.patch
>> From ab2a76a8e41d0c48cec8fc0f143e3c5df1b8b873 Mon Sep 17 00:00:00 2001
>> From: Alexander Strange <astrange at ithinksw.com>
>> Date: Wed, 17 Mar 2010 05:35:06 -0400
>> Subject: [PATCH 2/4] h264: Remove unused function argument
>> 
>> ---
>> libavcodec/h264_cabac.c |    8 ++++----
>> 1 files changed, 4 insertions(+), 4 deletions(-)
> 
> ok

Applied.