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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
