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

[Kareem Ergawy 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) &&
----------------
ergawy wrote:

This test was preventing induction variables, when `default(private)` is specified, from being added to the list of symbols privatized by the `DataSharingProcessor`.

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