[PATCH] D75591: [OpenMP] Add firstprivate as a default data-sharing attribute to clang

[Atmn Patel via Phabricator via cfe-commits]

atmnpatel added inline comments.


================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:2016-2023
+  if (DSAStack->getDefaultDSA() == DSA_firstprivate &&
+      VarsWithInheritedDSAType().count(D) == 0 &&
+      !DSAStack->hasExplicitDSA(
+          D, [](OpenMPClauseKind K) -> bool { return K == OMPC_firstprivate; },
+          Level, /*NotLastprivate=*/true) &&
+      Ty.getNonReferenceType()->isScalarType()) {
+    IsByRef = false;
----------------
@ABataev Is this what you mean?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75591