> On Sep 18, 2018, at 9:53 AM, Jim Jagielski <j...@jagunet.com> wrote:
> 
> 
> 
>> On Sep 18, 2018, at 9:33 AM, Ruediger Pluem <rpl...@apache.org> wrote:
>> 
>> 
>> 
>> On 09/17/2018 05:50 PM, j...@apache.org wrote:
>>> Author: jim
>>> Date: Mon Sep 17 15:50:19 2018
>>> New Revision: 1841078
>>> 
>>> URL: http://svn.apache.org/viewvc?rev=1841078&view=rev
>>> Log:
>>> Add in Atomics for 64bit ints
>>> 
>>> Added:
>>>   apr/apr/trunk/atomic/unix/builtins64.c   (with props)
>>>   apr/apr/trunk/atomic/unix/mutex64.c   (with props)
>>>   apr/apr/trunk/atomic/win32/apr_atomic64.c   (with props)
>>> Modified:
>>>   apr/apr/trunk/CHANGES
>>>   apr/apr/trunk/apr.dsp
>>>   apr/apr/trunk/include/apr_atomic.h
>>>   apr/apr/trunk/include/arch/unix/apr_arch_atomic.h
>>>   apr/apr/trunk/test/testatomic.c
>>> 
>> 
>> Hm, don't we miss some configure magic here?
> 
> 
> How so? All configure does is see if __sync_val_compare_and_swap, et.al. 
> exists

If it does, then it adds the 64bit versions. This is for Unix. For other 
platforms, either we know they do have 64bit atomics (eg: Windows) or else we 
use the portable versions of 64bit atomics.

Reply via email to