[FFmpeg-devel] [PATCH] nasm support

[Dave Yeo]

On 09/05/10 03:24 am, Reimar D?ffinger wrote:
> On Fri, Sep 03, 2010 at 07:05:58PM +0200, Reimar D?ffinger wrote:
>> On Fri, Sep 03, 2010 at 06:56:03PM +0200, Reimar D?ffinger wrote:
>>> Hello,
>>> this allows compiling FFmpeg asm code with nasm.
>>> Advantages:
>>> 1) nasm support OS/2 OMF format
>>> 2) there's a Cygwin package for nasm
>>> Tested on 64 bit Linux, compiles and passes tests.
>>> Obviously it would be a good idea for someone to test it
>>> on 32 bit...
>>
>> And this one is actually against current SVN.
>
> I took the liberty of applying the movd changes.
> Several of the asm files already used that syntax so I am hoping
> nobody has reason to flame me for that.
> Now I'd like to hear if there are any real objections to the fft_mmx.asm
> changes.
> For the configure part I need to know how OS/2 is now supposed
> to be handled.
> Note that I really don't intend for nasm support to cause any significant
> annoyance to those writing/maintaining the ASM.
> I just tried it out and the changes necessary so far simply didn't seem
> bad to me so I think we could try making a minimal effort to make users of
> OS/2 etc. happy.

This version of the patch builds and passes fate. Main difference is 
moving the section .text to a macro
Dave

-------------- next part --------------
A non-text attachment was scrubbed...
Name: nasm.diff
Type: text/x-patch
Size: 6401 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100905/f5f23bad/attachment.bin>