On 10/29/21 5:09 PM, yla...@apache.org wrote:
> Author: ylavic
> Date: Fri Oct 29 15:09:13 2021
> New Revision: 1894621
> 
> URL: http://svn.apache.org/viewvc?rev=1894621&view=rev
> Log:
> apr_atomic: Use __atomic builtins when available.
> 
> Unlike Intel's atomic builtins (__sync_*), the more recent __atomic builtins
> provide atomic load and store for weakly ordered architectures like ARM32 or
> powerpc[64], so use them when available (gcc 4.6.3+).
> 
> 
> Modified:
>     apr/apr/trunk/atomic/unix/builtins.c
>     apr/apr/trunk/atomic/unix/builtins64.c
>     apr/apr/trunk/atomic/unix/mutex.c
>     apr/apr/trunk/configure.in
> 

> 
> Modified: apr/apr/trunk/atomic/unix/mutex.c
> URL: 
> http://svn.apache.org/viewvc/apr/apr/trunk/atomic/unix/mutex.c?rev=1894621&r1=1894620&r2=1894621&view=diff
> ==============================================================================
> --- apr/apr/trunk/atomic/unix/mutex.c (original)
> +++ apr/apr/trunk/atomic/unix/mutex.c Fri Oct 29 15:09:13 2021
> @@ -89,7 +89,7 @@ static APR_INLINE apr_thread_mutex_t *mu
>  
>  APR_DECLARE(apr_status_t) apr_atomic_init(apr_pool_t *p)
>  {
> -    return apr__atomic_generic64_init(p);

Why is this no longer needed?

> +    return APR_SUCCESS;
>  }
>  
>  #endif /* APR_HAS_THREADS */
> 

Regards

RĂ¼diger

Reply via email to