[FFmpeg-devel] [PATCH] MMX for Win64

Ramiro Polla ramiro
Tue May 6 08:56:26 CEST 2008

matthieu castet wrote:
> Reimar D?ffinger wrote:
>> On Sun, May 04, 2008 at 05:05:47PM +0100, M?ns Rullg?rd wrote:
>>> "Zuxy Meng" <zuxy.meng at gmail.com> writes:
>>>> 2008/5/4 M?ns Rullg?rd <mans at mansr.com>:
>>>>> It's wrong to use size_t here.  There is no guarantee that size_t
>>>>> is at all suitable.  For instance, it could legally be 64 bits on a
>>>>> 32-bit system, or 32 bits on a 64-bit system.
>>>> What about intptr_t?
>>> That's guaranteed to be at least as wide as a pointer.  Since we're
>>> not dealing with pointers here, it is incorrect to use it.  It is
>>> possible to have 32-bit pointers and 64-bit long.
>> What is required here (and what long is "misused" for) is native
>> register size. And since all of the modified code is x86 specific,
>> pointers and native register contents are not really distinguishable.
>> Still, the proposed patch just hacks a few places and is not a proper
>> solution.
> And since there already conditional code in libavutil/x86_cpu.h for 
> X86_32 and X86_64, it could be trivial to define here the type we want.

Attached patch passes regression tests on mingw32 and mingw64.

Ramiro Polla
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: amd64.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080506/2b8c971e/attachment.txt>

More information about the ffmpeg-devel mailing list