[flang-commits] [flang] [flang][OpenMP] Try to unify induction var privatization for sequential loops inside parallel regions. (PR #91116)

[Leandro Lupori via flang-commits]

================
@@ -344,7 +353,6 @@ void DataSharingProcessor::collectSymbols(
     assert(curScope && "couldn't find current scope");
     if (isPrivatizable(*sym) && !symbolsInNestedRegions.contains(sym) &&
         !privatizedSymbols.contains(sym) &&
-        !sym->test(Fortran::semantics::Symbol::Flag::OmpPreDetermined) &&
----------------
luporl wrote:

> > more accurate and to simplify defaultPrivatize and implicitPrivatize.
> 
> Can you elaborate on what you mean by simplify?

Before #85989, `defaultSymbols` would have other unrelated symbols, that would be filtered at `defaultPrivatize` only. The filter part was moved to `collectSymbols`, which simplified `defaultPrivatize` and `implicitPrivatized` and allowed `symbolsInNestedRegions` to be removed.

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