[Openmp-commits] [PATCH] D84557: [OpenMP][Tests] Enable nvptx64 testing for most libomptarget tests

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a subscriber: hans.
jdoerfert added a comment.

In D84557#2178019 <https://reviews.llvm.org/D84557#2178019>, @protze.joachim wrote:

> Thanks for reviewing.
>
> At the time of the commit 9 tests are failing during offloading to V100:
>
> Broken because of D83963 <https://reviews.llvm.org/D83963>, @jdenny proposed a fix, which works for me:
>
>   libomptarget :: mapping/alloc_fail.c
>   libomptarget :: mapping/present/target.c
>   libomptarget :: mapping/present/target_data.c
>   libomptarget :: mapping/present/target_enter_data.c
>   libomptarget :: mapping/present/target_exit_data.c
>   libomptarget :: mapping/present/zero_length_array_section.c
>   libomptarget :: mapping/present/zero_length_array_section_exit.c
>
> Broken since D68100 <https://reviews.llvm.org/D68100>, solution seems to be unclear:
>
>   libomptarget :: mapping/declare_mapper_target.cpp
>
> Fix is discussed in D84470 <https://reviews.llvm.org/D84470>, patch works for me:
>
>   libomptarget :: offloading/target_depend_nowait.cpp

Thanks. Could you ask @hans to include this in the release (via your bug) if you haven't already?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84557