[PATCH] D69204: [OpenMP 5.0] - Extend defaultmap

[Chi Chun Chen via Phabricator via cfe-commits]

cchen marked 12 inline comments as done.
cchen added inline comments.


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:1978
+           DSAStack->getDefaultDMIBAtLevel(Level, OMPC_DEFAULTMAP_scalar) ==
+           DMIB_firstprivate) ||
           DSAStack->hasExplicitDSA(
----------------
ABataev wrote:
> If the scalars are mapped as firstprivates by default, they must be captured by value.
I somehow forgot about this, will fix this soon.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69204