[FFmpeg-devel] [PATCH 2/2] Remove ff_inverse from libavcodec.

[Måns Rullgård]

Uoti Urpala <uoti.urpala at pp1.inet.fi> writes:

> On Thu, 2008-10-02 at 18:57 +0200, Diego 'Flameeyes' Petten? wrote:
>> The previous patch added a copy of ff_inverse in libavutil, this patch
>> removes the copy present in libavcodec, making it rely on the one
>> present in libavutil.
>
> Using a symbol from another library is slower (a symbol from the same
> library is at a constant offset, a symbol from another library requires
> lookup through offset table). Since the whole point of FASTDIV is speed
> this could matter.

Is this true also for read-only data?  Doesn't that generate a copy
relocation rather than a GOT indirection?  If in doubt, check the
disassembly.

-- 
M?ns Rullg?rd
mans at mansr.com