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

[Shilei Tian via Phabricator via Openmp-commits]

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