[FFmpeg-devel] [PATCH] MMX for Win64
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
> 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.
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
More information about the ffmpeg-devel