[FFmpeg-devel] [PATCH] avcodec/motion_est: remove duplicate function

Michael Niedermayer michael at niedermayer.cc
Sun Jan 20 22:47:52 EET 2019


On Sun, Jan 20, 2019 at 07:35:18PM +0100, Marton Balint wrote:
> 
> 
> On Sun, 20 Jan 2019, Michael Niedermayer wrote:
> 
> >On Sat, Jan 19, 2019 at 11:48:39PM +0100, Marton Balint wrote:
> >>Signed-off-by: Marton Balint <cus at passwd.hu>
> >>---
> >> libavcodec/motion_est.c          |  4 +--
> >> libavcodec/motion_est_template.c | 62 +---------------------------------------
> >> 2 files changed, 3 insertions(+), 63 deletions(-)
> >
> >please check if the compiler optimizes the size constant out after this
> >change
> 
> The compiler inlines the function before and after the change as well. I
> can't see notable changes in the disassembly of interlaced_search and
> h263_mv4_search. Is this enough, or something else should be checked? I am
> not sure how...

If the inlined code is used with only one size value then its probably ok.
you could also put some marker with asm() in the code where size is used
and then look at the .s file generated if the size is optimized out or
still read as a variable

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

"Nothing to hide" only works if the folks in power share the values of
you and everyone you know entirely and always will -- Tom Scott

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20190120/953ca0fb/attachment.sig>


More information about the ffmpeg-devel mailing list