[flang-commits] [flang] [llvm] [flang][OpenMP] Try to unify induction var privatization for OMP regions. (PR #91116)

[Leandro Lupori via flang-commits]

================
@@ -289,12 +290,34 @@ void DataSharingProcessor::collectSymbolsInNestedRegions(
        eval.getNestedEvaluations()) {
     if (nestedEval.hasNestedEvaluations()) {
       if (nestedEval.isConstruct())
-        // Recursively look for OpenMP constructs within `nestedEval`'s region
----------------
luporl wrote:

I would leave this comment. It helps to understand what this part of the code is doing, that we only continue looking for OpenMP constructs recursively for non-OpenMP constructs. Otherwise `converter.collectSymbolSet` can be used to collect the remaining symbols.

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