[Mesa-dev] [PATCH] gallivm: fix a missing argument to CreateAtomicCmpXchg

Brian Paul brianp at vmware.com
Fri Aug 2 03:08:41 UTC 2019


On 08/01/2019 04:56 PM, Charmaine Lee wrote:
> This patch fixes a missing argument to CreateAtomicCmpXchg for older
> version of LLVM.
> ---
>   src/gallium/auxiliary/gallivm/lp_bld_misc.cpp | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp b/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
> index 79d1029..8205d24 100644
> --- a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
> +++ b/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp
> @@ -831,6 +831,7 @@ LLVMValueRef LLVMBuildAtomicCmpXchg(LLVMBuilderRef B, LLVMValueRef Ptr,
>      /* LLVM 3.8 doesn't have a second ordering and uses old SynchronizationScope enum */
>      return llvm::wrap(llvm::unwrap(B)->CreateAtomicCmpXchg(llvm::unwrap(Ptr), llvm::unwrap(Cmp),
>                                                             llvm::unwrap(New), mapFromLLVMOrdering(SuccessOrdering),
> +                                                          mapFromLLVMOrdering(FailureOrdering),
>                                                             SingleThread ? llvm::SynchronizationScope::SingleThread : llvm::SynchronizationScope::CrossThread));
>   }
>   #endif
> 

Reviewed-by: Brian Paul <brianp at vmware.com>


More information about the mesa-dev mailing list