[Openmp-commits] [openmp] 7fd6d2b - [OpenMP] remove an erroneous assert on the location argument

Vadim Paretsky via Openmp-commits openmp-commits at lists.llvm.org
Fri May 12 15:41:05 PDT 2023


Author: Vadim Paretsky
Date: 2023-05-12T15:40:37-07:00
New Revision: 7fd6d2babf4dcaa24f24da18cd0a858fa89579e8

URL: https://github.com/llvm/llvm-project/commit/7fd6d2babf4dcaa24f24da18cd0a858fa89579e8
DIFF: https://github.com/llvm/llvm-project/commit/7fd6d2babf4dcaa24f24da18cd0a858fa89579e8.diff

LOG: [OpenMP] remove an erroneous assert on the location argument

The 'loc' argument is optional, and some compilers (e.g. MSVC) do no supply it.

Differential Revision: https://reviews.llvm.org/D148393

Added: 
    

Modified: 
    openmp/runtime/src/kmp_runtime.cpp

Removed: 
    


################################################################################
diff  --git a/openmp/runtime/src/kmp_runtime.cpp b/openmp/runtime/src/kmp_runtime.cpp
index a0a72f163e7c5..54d280534a33f 100644
--- a/openmp/runtime/src/kmp_runtime.cpp
+++ b/openmp/runtime/src/kmp_runtime.cpp
@@ -8827,7 +8827,6 @@ __kmp_determine_reduction_method(
 
   int team_size;
 
-  KMP_DEBUG_ASSERT(loc); // it would be nice to test ( loc != 0 )
   KMP_DEBUG_ASSERT(lck); // it would be nice to test ( lck != 0 )
 
 #define FAST_REDUCTION_ATOMIC_METHOD_GENERATED                                 \


        


More information about the Openmp-commits mailing list