[FFmpeg-devel] [PATCH 2/3] avcodec/kmvc: Move commonly used variables to the front of the context

[Andreas Rheinhardt]

Tomas Härdin:
> mån 2021-03-22 klockan 03:06 +0100 skrev Andreas Rheinhardt:
>> Reduces codesize because the offset in pointer+offset addressing
>> requires less bytes to encode. Reduces the size of .text from 8871B
>> to 8146B (GCC 10, -O3, x64).
>>
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
>> ---
>>  libavcodec/kmvc.c | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/libavcodec/kmvc.c b/libavcodec/kmvc.c
>> index 8d9f0a9693..dd1ae05f2d 100644
>> --- a/libavcodec/kmvc.c
>> +++ b/libavcodec/kmvc.c
>> @@ -44,12 +44,12 @@
>>  typedef struct KmvcContext {
>>      AVCodecContext *avctx;
> 
> Couldn't this be removed too? Doesn't have to hold up this patch of
> course
> 
It is currently used for log messages; such usage is fairly common, but
it can of course be changed. Don't know if it is beneficial though.

- Andreas