[Openmp-commits] [PATCH] D19878: Use C++11 atomics for ticket locks implementation
Jim Cownie via Openmp-commits
openmp-commits at lists.llvm.org
Wed May 4 02:51:06 PDT 2016
jcownie added a subscriber: jcownie.
jcownie added a comment.
I have only made a top-level scan, so this needs more review. However,
__kmp_baker_check should look like this
static kmp_uint32
__kmp_bakery_check(void *value, kmp_uint32 checker)
{
return (std::atomic_load( (std::atomic<unsigned> *)value ) == checker);
}
Rather than having a test then returning TRUE or FALSE!
It would also be nice to reduce some of the code replication (e.g. kmp_wait_yield_4_ptr is identical to kmp_wait_yield_4 apart from the argument type [MIssing leading underscores since they cause underlining!] ). Either we should use a template (if we accept that this code is really C++ now), or even a macro for the whole function that is invoked with the function name and type is better than having the whole code duplicated.
Repository:
rL LLVM
http://reviews.llvm.org/D19878
More information about the Openmp-commits
mailing list