[Openmp-commits] [PATCH] D65340: [OpenMP][libomptarget] Add support for close map modifier

Gheorghe-Teodor Bercea via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Aug 5 11:29:53 PDT 2019


gtbercea marked an inline comment as done.
gtbercea added inline comments.


================
Comment at: libomptarget/test/unified_shared_memory/close_modifier.c:28-30
+  // Manual registration of requires flags for Clang versions
+  // that do not support requires.
+  __tgt_register_requires(8);
----------------
Hahnfeld wrote:
> Unfortunately, I don't think that's enough for this test to pass with older versions of Clang because it didn't support code generation for `close`. Did you test this?
I don't think it will either but I'm not sure how we can actually make it work. I think there's not much more this test can do. To make this work probably requires a more forgiving check in the compiler.


Repository:
  rOMP OpenMP

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

https://reviews.llvm.org/D65340





More information about the Openmp-commits mailing list