[Openmp-commits] [PATCH] D34701: [openmp-target-tests] OpenMP 4.5 Target data test cases

sunita chandrasekaran via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri Aug 4 07:31:43 PDT 2017

schandra added a comment.

In https://reviews.llvm.org/D34701#828682, @Hahnfeld wrote:

> In https://reviews.llvm.org/D34701#823166, @schandra wrote:
> > Hi Jonas
> >
> > Did you have further comments on this commit?
> >
> > Thanks 
> > Sunita
> Sorry that it's taken so long! Still not sure about the Fortran test as clang will currently not be able to handle this.
> Additionally, these tests currently won't pass because `omp_is_initial_device()` will return `1` when offloading to the host, correct? See https://reviews.llvm.org/D35719 why we had to disable libomptarget for 5.0.
> Find some more comments inline. In general, the tests should be brought to a consistent coding style (braces, end-comments, comment style and so on).

Great!! Thanks Jonas, we will fix these shortly. 


More information about the Openmp-commits mailing list