[Openmp-commits] [PATCH] D40358: Use hyperbarrier by default on all architectures

[Andrey Churbanov via Phabricator via Openmp-commits]

AndreyChurbanov added a comment.

The idea was that 32-bit machines will probably have small number of cores (2, or 4, or ...).  Then hyper barrier can have bigger overhead.  Can you check if 2 or 4 threads work faster on hyper barrier comparing to linear?   If not, then maybe the condition could be fixed in different way, e.g. adding Power arch to the x86_64, leaving linear barrier for 32-bit archs.

BTW, the comments "hyper2: C78980" could be safely removed I think.  This is some very old info that says nothing nowadays (at least to me:).


https://reviews.llvm.org/D40358