[Openmp-commits] [PATCH] D42427: Fix broken OpenMP runtime test cases for Windows

[Hans Wennborg via Phabricator via Openmp-commits]

hans added a comment.

In https://reviews.llvm.org/D42427#987732, @omalyshe wrote:

> In https://reviews.llvm.org/D42427#987666, @hans wrote:
>
> > This works for me if the NEEDS are replaced with REQUIRES.
> >
> > ompt/misc/control_tool_no_ompt_support.c still fails though:
>
>
> Did you replace NEEDS with REQUIRES in this test as well? (REQUIRES: omp50)


There was no NEEDS line in that file. Does omp50 get defined anywhere? Grepping the OpenMP repository for it yields no matches.


https://reviews.llvm.org/D42427