[Openmp-commits] [PATCH] D49857: [OpenMP] Implement GOMP doacross compatibility
Andrey Churbanov via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Tue Jul 31 02:19:06 PDT 2018
AndreyChurbanov added a comment.
In https://reviews.llvm.org/D49857#1181735, @Hahnfeld wrote:
> In https://reviews.llvm.org/D49857#1180955, @jlpeyton wrote:
>
> > I can create a typical 2D wavefront testcase that uses the actual doacross clauses, and we can omit gcc from kmp_doacross_check.c. Does this suffice?
>
>
> I think it's good to have an end-to-end test now that the library has interfaces for all compilers.
>
> For `kmp_doacross_check.c` another possibility would be to guard the call to `__kmpc_doacross_fini` in `#ifndef __GNUC__`. I'm not sure if this is elegant though...
Intel compiler defines __GNUC__ to express compatibility I guess, so it is safer to disable the whole test.
Repository:
rOMP OpenMP
https://reviews.llvm.org/D49857
More information about the Openmp-commits
mailing list