[Openmp-commits] [PATCH] Initial Dynamic Lock Dispatch Code in OpenMP

[hfinkel at anl.gov]

> This change is rather large, but currently is disabled. One can turn it on by changing the value of KMP_USE_DYNAMIC_LOCK in kmp_os.h to 1


Why is it disabled by default? Are you planning to enable this by default?

>   Our performance results for micro-benchmarks show that the runtime selection of lock implementations does not add any noticeable overhead. We also show that using an appropriate runtime hint can improve application performance.


Looking toward the hopefully-not-so-distant future when we have regression tests for this library, could you please commit these microbenchmarks to the repository somewhere? This will help me to review this patch, and can also serve as the basis for tests for this feature.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7631

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/