Hi,
>iam a bit affraid this could cause a slowdown. did you check what
effect this change has
>on compilers supporting the code currently ?
No, sorry I am not able to quantify the impact of this change at this
point. At least not with tangible numbers, with a quick transcoding run
(Linux x86, gcc4.6) I didn't notice a (significant) difference.
Maybe you like the attached revised patch better - having two functions,
one that is attributed inline and one for external reference.
>changing all MANGLE() to "m" will break compilation on several
compilers
Yeah, I understand this. And cluttering the code with #ifdef's would be
a mess I suppose. OK, then can't help it.
>isnt icc able to optimize this out with some command line switches ?
>under linux ICC is able to do such elimination of calls ...
>if it really doesnt work its maybe worth to report to intel if it hasnt
been reported yet. Intel
>is likely interrested to improve ICCs dead code elimination.
Good idea, I didn't think of that. I'll check this out. I seem to
remember that it is a compatibility issue (compatibility with the
Microsoft compiler).
Regards,
Juergen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-revised-patch-for-ff_get_mb_score-ff_epzs_motion_sea.patch
Type: application/octet-stream
Size: 5349 bytes
Desc: 0004-revised-patch-for-ff_get_mb_score-ff_epzs_motion_sea.patch
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120614/a2bba6ca/attachment.obj>