Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2019-01-04 Thread Manoj Gupta
Thanks a lot :-) On Thu, Jan 3, 2019 at 1:53 PM Michael Niedermayer wrote: > > On Wed, Jan 02, 2019 at 06:29:22PM -0800, Manoj Gupta wrote: > > On Wed, Jan 2, 2019 at 2:33 PM Michael Niedermayer > > wrote: > > > > > > On Wed, Jan 02, 2019 at 10:12:33AM -0800, Manoj Gupta wrote: > > > > On Mon, D

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2019-01-03 Thread Michael Niedermayer
On Wed, Jan 02, 2019 at 06:29:22PM -0800, Manoj Gupta wrote: > On Wed, Jan 2, 2019 at 2:33 PM Michael Niedermayer > wrote: > > > > On Wed, Jan 02, 2019 at 10:12:33AM -0800, Manoj Gupta wrote: > > > On Mon, Dec 31, 2018 at 8:31 AM Michael Niedermayer > > > wrote: > > > > > > > > On Fri, Dec 28, 20

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2019-01-02 Thread Manoj Gupta
On Wed, Jan 2, 2019 at 2:33 PM Michael Niedermayer wrote: > > On Wed, Jan 02, 2019 at 10:12:33AM -0800, Manoj Gupta wrote: > > On Mon, Dec 31, 2018 at 8:31 AM Michael Niedermayer > > wrote: > > > > > > On Fri, Dec 28, 2018 at 03:12:53PM -0800, Manoj Gupta wrote: > > > > Hi All, > > > > > > > > I

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2019-01-02 Thread Michael Niedermayer
On Wed, Jan 02, 2019 at 10:12:33AM -0800, Manoj Gupta wrote: > On Mon, Dec 31, 2018 at 8:31 AM Michael Niedermayer > wrote: > > > > 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 > > > com

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2019-01-02 Thread Manoj Gupta
On Mon, Dec 31, 2018 at 8:31 AM Michael Niedermayer wrote: > > 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 f

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2018-12-31 Thread Michael Niedermayer
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 func

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2018-12-30 Thread Carl Eugen Hoyos
2018-12-29 17:17 GMT+01:00, Manoj Gupta : > On Sat, Dec 29, 2018 at 2:32 AM Carl Eugen Hoyos wrote: >> >> 2018-12-29 0:12 GMT+01:00, Manoj Gupta : >> >> > I recently had a problem building ffmpeg for AArch64 where lld >> > linker complained about text relocations in readonly segment. >> > The foll

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2018-12-29 Thread Manoj Gupta
On Sat, Dec 29, 2018 at 2:32 AM Carl Eugen Hoyos wrote: > > 2018-12-29 0:12 GMT+01:00, Manoj Gupta : > > > 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 referrin

Re: [FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2018-12-29 Thread Carl Eugen Hoyos
2018-12-29 0:12 GMT+01:00, Manoj Gupta : > 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. Do you know why this

[FFmpeg-devel] [PATCH] libavcodec: Remove dynamic relocs from aarch64/h264idct_neon.S

2018-12-28 Thread Manoj Gupta
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: