[flang-commits] [flang] [flang][OpenMP] Fix construct privatization in default clause (PR #72510)

[via flang-commits]

================
@@ -426,14 +425,10 @@ void DataSharingProcessor::collectSymbols(
                              /*collectSymbols=*/true,
                              /*collectHostAssociatedSymbols=*/true);
   for (Fortran::lower::pft::Evaluation &e : eval.getNestedEvaluations()) {
-    if (e.hasNestedEvaluations())
+    if (e.hasNestedEvaluations() && !e.isConstruct())
----------------
NimishMishra wrote:

Additionally, for constructs like `critical` not having an explicit default clause, the lowered IR is in line with the correct symbols that the construct should be using.

https://github.com/llvm/llvm-project/pull/72510