Re: [PATCH] Windows x64 [repost]

On Fri, Jan 1, 2010 at 20:45, Magnus Hagander <magnus@hagander.net> wrote:
> On Fri, Dec 4, 2009 at 11:42, Tsutomu Yamada <tsutomu@sraoss.co.jp> wrote:
>>
>> 2) use appropriate macro and datatypes for Windows API.
>>   enables more than 32bits shared memory.
>
> Are you sure this one should use __noop, and not __nop?
>
> __noop: http://msdn.microsoft.com/en-us/library/s6btaxcs.aspx
> __nop: http://msdn.microsoft.com/en-us/library/aa983381.aspx
>
> I think __nop is what we want?
>
> Also, that turns it into "nop" and not "rep nop", no?

I did some more research, and __nop() is at least closer than
__noop(), but it's still not the same.


> Should we perhaps instead use __yield, per:
> http://msdn.microsoft.com/en-us/library/2b2h26kx.aspx

On further reading, __yield() is only available on Itanium.


-- Magnus HaganderMe: http://www.hagander.net/Work: http://www.redpill-linpro.com/