[Mesa-dev] [PATCH] util/u_atomic: Add new macro p_atomic_add

Jose Fonseca jfonseca at vmware.com
Mon Feb 9 08:19:19 PST 2015


On 06/02/15 22:39, Carl Worth wrote:
> On Fri, Feb 06 2015, Aaron Watry wrote:
>> Ignore me if this is a stupid question, but should those both be
>> sizeof(short)?  I'd expect the first to be sizeof(char).
>
> Not a stupid question. That was a copy-and-paste (kill-and-yank ?) bug
> of mine.
>
> Thanks for your attention to detail. I've fixed this in my tree.
>
> -Carl

Hi Carl,

Just one more tweak to InterlockedExchangeAdd64 as per patch attached. 
(The 64bit intrisicis are only available on 64-bits, but the 
non-intrisinc version is available everywhee.)

With that u_test_atomic builds and passes for me both on 32 and 64bits.

Sorry for the delay. And thanks for your help in keeping MSVC support on 
par.

Jose

-------------- next part --------------
A non-text attachment was scrubbed...
Name: msvc.patch
Type: text/x-patch
Size: 1018 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20150209/4ae4c9ba/attachment.bin>


More information about the mesa-dev mailing list