Aside from the minor nit below this patch LGTM. Thanks a lot for all the changes @HsiangKai.

Thanks for the update @HsiangKai.

Apologies we didn't identify this earlier but with the change of the mask register layout (MLEN=1) the overlap constraints involving the mask register were modified:

Hi @HsiangKai, thanks for the patch. So far everything looks good aside from a couple of minor nits.

I'm not a reviewer but the patch LGTM, thanks for all the changes @HsiangKai.

I've added a couple more inline comments.

Hi Kai, I've added an inline comment regarding constraint validation:

Thanks for addressing my comments @HsiangKai. After a closer look I've added some more inline comments, and some general ones too:

Thanks for updating the patch to the stable v0.8 release (previously version 0.8-draft-20191213). I've added some inline comments.

I believe I'm having a problem with Archer when I run the OpenMP tests with an architecture that doesn't support TSan yet:

Hi, thank you for this patch!

I've tested this patch with current upstream LLVM and Clang on a HiFive Unleashed board and all tests are passing, including OMPT. I don't have commit access yet, @Hahnfeld would you mind commiting this for me? Thank you!

Now that D60456 has landed we believe this need not be WIP anymore.

Thanks @Hahnfeld for the review. I've just updated the patch.

Remove reset of exit_frame pointer for OMPT since it is now done at call-site and fix return address computation macro used for some tests (as the generated code has changed).

Thank you @jlpeyton!

I've noticed some tests in libomp still have the REQUIRES: openmp-4.5 or REQUIRES: openmp-4.0 constraints. Given that the REQUIRES: openmp-5.0 have been removed I am wondering whether this may be an oversight. Those tests are currently not run, as lit considers them unsupported.

Thanks for the quick review @Hahnfeld, I've just replied to your inline comments and updated the patch!

Update print_possible_return_addresses macro in runtime/test/ompt/callback.h to handle uncompressed instructions, as pointed out by @Hahnfeld.

I'm glad it helps, thank you for your time!

Great, thank you for reviewing!

Thank you. I've updated the test using 'update_llc_test_checks.py' and I've added a comment as suggested.

Thank you Alex. I've updated the test name and I've added a comment.
Hope it looks better now.

Ping.