[Openmp-commits] [PATCH] D60578: [OPENMP][NVPTX]Fix dynamic scheduling in L2+ SPMD parallel regions.

Johannes Doerfert via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Apr 11 15:08:38 PDT 2019


jdoerfert added a comment.

In D60578#1463461 <https://reviews.llvm.org/D60578#1463461>, @ABataev wrote:

> In D60578#1463452 <https://reviews.llvm.org/D60578#1463452>, @jdoerfert wrote:
>
> > In D60578#1463411 <https://reviews.llvm.org/D60578#1463411>, @gtbercea wrote:
> >
> > > LGTM
> >
> >
> > Is there a way we can test this?
>
>
> It is tested in the internal testsuite, don't know when it is going to be committed to trunk


There are two problems:

1. The internal testsuite did run before this patch, right? So it is unclear what that means.
2. Changes done upstream might break this without us noticing for a while and without being able to know apriory.

Why don't we have unit tests here or in the llvm-test suite?


Repository:
  rOMP OpenMP

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

https://reviews.llvm.org/D60578





More information about the Openmp-commits mailing list