[FFmpeg-devel] [PATCH] Workaround Mac-OS x86_64 dynamic loader bug for Yasm generated code

[Reimar Döffinger]

On Fri, Feb 06, 2009 at 09:00:50PM -0500, Alexander Strange wrote:
> 
> On Feb 4, 2009, at 3:53 PM, Art Clarke wrote:
> 
> > On Wed, Feb 4, 2009 at 12:27 PM, Art Clarke <aclarke at xuggle.com>  
> > wrote:
> >> See this chain for details on the problem:
> >> http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2009-February/061189.html
> >>
> >> Tested on Mac and Linux in 32 and 64 bit builds and everything  
> >> seems to work.
> >
> > My bad; I attached an earlier patch.  Correct patch is attached (this
> > one doesn't set alignment to 16 after change)
> 
> This patch seems correct for all systems to me, not just macho64,  
> since dispatch_tab* is data. But I don't have another OS to test  
> removing the %ifidn on.

You misunderstood the purpose of the patch, mach targets can't use
.rodata for anything that needs alignment because something is broken,
thus most is placed in .text.
But relocations are not allowed in .text, so this one must be in .rodata
(luckily it has no alignment requirements).