[FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S
michael at niedermayer.cc
Mon Dec 31 18:30:53 EET 2018
On Fri, Dec 28, 2018 at 03:12:53PM -0800, Manoj Gupta wrote:
> Hi All,
> I recently had a problem building ffmpeg for AArch64 where lld linker
> complained about text relocations in readonly segment. The following
> patch fixes the linker complains by referring to a local label instead
> of function name.
> This is similar in nature as the following previous commits:
> libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S
> Some of the assembly functions e.g. ff_h264_idct_dc_add_neon
> has code like:
> movrel x14, X(ff_h264_idct_add_neon)
> Linker cannot resolve them fully at link time and emits dynamic
> Use explicit labels instead so that no dynamic relocations are
> needed at all.
> This avoids lld complains about text relocations.
> For background, see https://crbug.com/917919
> Signed-off-by: Manoj Gupta <manojgupta at chromium.org>
> libavcodec/aarch64/h264idct_neon.S | 20 ++++++++++++--------
> 1 file changed, 12 insertions(+), 8 deletions(-)
Has this been tested on all common aarch64 platforms ?
Also git doesnt separate the mail from the commit message and would
put the wholw email as the commit message, i assume that is not
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Modern terrorism, a quick summary: Need oil, start war with country that
has oil, kill hundread thousand in war. Let country fall into chaos,
be surprised about raise of fundamantalists. Drop more bombs, kill more
people, be surprised about them taking revenge and drop even more bombs
and strip your own citizens of their rights and freedoms. to be continued
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: not available
More information about the ffmpeg-devel