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

[via flang-commits]

================
@@ -268,21 +268,39 @@ void DataSharingProcessor::insertLastPrivateCompare(mlir::Operation *op) {
   firOpBuilder.restoreInsertionPoint(localInsPt);
 }
 
+void DataSharingProcessor::collectSymbolsInNestedRegions(
+    Fortran::lower::pft::Evaluation &eval,
+    Fortran::semantics::Symbol::Flag flag,
+    llvm::SetVector<const Fortran::semantics::Symbol *>
+        &symbolsInNestedRegions) {
+  for (Fortran::lower::pft::Evaluation &nestedEval :
+       eval.getNestedEvaluations()) {
+    if (nestedEval.hasNestedEvaluations()) {
----------------
NimishMishra wrote:

I don't think so. All such constructs should have a block associated with them, and thus will be lowered as nested evals

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