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

Paul Osmialowski via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Nov 7 05:37:47 PST 2018


pawosm01 added a comment.

I need to contact someone who introduced KMP_REVERSE_HYPER_BAR implementation (unfortunately, it was added here in a huge collective commit years ago, Tue Oct 7 16:25:50 2014).
I have a piece of proprietary code that stopped to fail on AArch64 after I reverted https://reviews.llvm.org/D40358. Digging deeper I found out that it's not the problem with hyperbariers per se. As I undefined KMP_REVERSE_HYPER_BAR, the problem went away. What else I found is that the problem can be reproduced on x86_64 too, only less frequent (once per ```for i in `seq 1 1000`; do``` execution). As I'd have to spend a time to figure out the logic behind it and verify its validity, in the meantime, can someone familiar with this code have a look at it and confirm reverse implementation is sane?


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D40358





More information about the Openmp-commits mailing list