[PATCH] D101849: [OpenMP][NFC] Refactor Clang OpenMP tests using update_cc_test_checks

Tue May 18 11:31:41 PDT 2021

jdoerfert added a comment.

In D101849#2766411 <https://reviews.llvm.org/D101849#2766411>, @mikerice wrote:

> In D101849#2764825 <https://reviews.llvm.org/D101849#2764825>, @ABataev wrote:
>
>> No, it is not required. Most probably, needed to simplify test checks, nothing else.
>
> Thanks.  I'd like to remove the "REQUIRES: x86-registered-target", the -O1 for CHECK6,10, and regenerate the CHECK lines.  Unfortunately I am seeing this mangling issue (https://bugs.llvm.org/show_bug.cgi?id=49767) when running the script. @ggeorgakoudis, how did you get past this to generate the CHECK lines in your change?

The workaround is to check if the assertion would trigger and just return from the function instead.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101849/new/

https://reviews.llvm.org/D101849