gallivm: fix a missing argument to CreateAtomicCmpXchg

Submitted by Jose Fonseca on Aug. 2, 2019, 8:50 a.m.

Details

Message ID SN2PR05MB2461889EBC8C044A50AD6D1BB6D90@SN2PR05MB2461.namprd05.prod.outlook.com
State New
Headers show
Series "gallivm: fix a missing argument to CreateAtomicCmpXchg" ( rev: 2 ) in Mesa

Not browsing as part of any series.

Commit Message

Jose Fonseca Aug. 2, 2019, 8:50 a.m.
Looks good to me.

Reviewed-by: Jose Fonseca <jfonseca@vmware.com>

Patch hide | download patch | download mbox

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