[Openmp-commits] [PATCH] D85342: [OpenMP] Fix `target data` exit for array extension

[Joel E. Denny via Phabricator via Openmp-commits]

jdenny added inline comments.


================
Comment at: openmp/libomptarget/src/omptarget.cpp:509
+                                              UpdateRef, IsHostPtr,
+                                              /*MustContain=*/true);
     if (!TgtPtrBegin && (DataSize || HasPresentModifier)) {
----------------
Ah, I broke reference count decrements for implicit mappings.  I think it should be `true` -> `!IsImplicit`.  I'll work on another patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85342