[Openmp-commits] [PATCH] D159119: [OpenMP] Enable optimization level for memory_manager.cpp OpenMP test.

Shilei Tian via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Thu Aug 31 08:42:23 PDT 2023


tianshilei1992 added a comment.

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

> The test actually has an OpenMP issue:
>
> The memory is allocated on device 0, but no device is specified for the target region. I think, all target constructs should explicitly have a `device(0)` clause. At the moment, the test relies on implementation-defined behavior.

I don’t think that is the cause of the issue. The default device is 0, at least for OpenMP.


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

https://reviews.llvm.org/D159119



More information about the Openmp-commits mailing list