[Openmp-commits] [PATCH] D51687: [libomptarget-nvptx] Add testing infrastructure

[Joachim Protze via Phabricator via Openmp-commits]

protze.joachim added a comment.

In https://reviews.llvm.org/D51687#1230375, @gtbercea wrote:

> Considering your comment in the description about requiring latest Clang perhaps you should revisit this patch: https://reviews.llvm.org/D46842


https://reviews.llvm.org/D46842 only helps if the OpenMP runtime is not built stand-alone. 
For the OpenMP runtime tests, we take care that the tests also work with other compatible compilers, and also with older versions.

We might configure different behavior for stand-alone and in-llvm-tree builts.
Considering the necessity of a GPU supporting the features, it makes sense to have a separate target for these tests and executing them manually.


https://reviews.llvm.org/D51687