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

Giorgis Georgakoudis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 18 12:45:19 PDT 2021


ggeorgakoudis 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?

I built without asserts :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101849



More information about the cfe-commits mailing list