Follow suggestions by jlpeyton. Thanks a lot.

I couldn't find any documents on the web. Is there any existing materials?

Thanks for the suggestion. I tried if lit's regex can be used but got no luck. So, I decided to follow the advice.
I think it is better to separate patches and here is the patch to run barrier/reduction test for each barrier : D123359

Hi
I saw check-openmp hang on both x86 and arm64. Adding timeout value to lit option gives me the following result:

I checked the runtime tests and found that openmp/runtime/test/barrier/omp_barrier.c already triggers different combinations of barriers. Do the combinations cover all configurations you tried manually?

Not really.
There are currently 5 barriers in libomp: linear, tree, hyper(default), hier, dist.
With manually changing barriers with this patch, I don't see any problems with linear, tree and hyper while found some in hier and dist (incl. the problems addressed by other patches I submitted).

Can you add extra run lines to openmp/runtime/test/ompt/synchronization/reduction/*_reduce.c in order to trigger the different barrier implementations?

I think, in addition to manually selecting the barrier implementation, we should add additional run lines to specific tests that trigger the different barrier implementations. I added a related, OMPT-specific comment to D123193.

While testing this, I found some minor problems in hier and dist barrier. Here are the patches: D123193, D123194, D123195

I noticed this when running check-openmp with dist barrier by using https://reviews.llvm.org/D122645
btw, it seems parallel tests run does not work when using dist barrier, but I couldn't figure out the reason.

I noticed this when running check-openmp with dist barrier by using https://reviews.llvm.org/D122645

I noticed this when running check-openmp with hier barrier by using https://reviews.llvm.org/D122645

Update name to use CHECK_OPENMP_ENV and also update README

Thanks for clarifications, I'm ok with CHECK_OPENMP_ENV.
If anyone has a different suggestion, please let me know.

Obviously I didn't add proper ifdef close and compile on non-arm64 architecture fails for current version. I will fix that.

Sorry, I wrote wrong location for kernel driver. I updated summary to correct one.

Friendly ping as this seems to have forgotten. Thanks.

Gentle ping, or can anyone help me to commit this? Thanks.

Thank you very much. Could I also ask you to commit this patch as I don't have right?

I'm not sure who is the code owner of this. Could you please review this as I see you have previously review similar commit?

As I don't have access right, would you mind committing this instead of me?

Hello, this is the first time I submitted a patch and please let me know if I misunderstood something.