[flang-commits] [flang] [flang][OpenMP] Try to unify induction var privatization (PR #91116)

[Kiran Chandramohan via flang-commits]

kiranchandramohan wrote:

Could you check why the CI is failing?

>  Try to unify induction var privatization

Could you clarify that this is for sequential loops inside parallel region? We have a separate case for induction variables of worksharing loops. 


> If you agree with this change, we will be able to switch delayed privatization by default (see #90945). 

We should enable delayed privatization for all constructs that are currently supported before switching it on. Otherwise we will regress on existing enabled applications. This will involve adding translation support for constructs for which there is support already.
1. sections, single, worksharing loop, simd (?), task (only works for non-deferred execution).
2. In some cases a barrier has to be added after first-privatisation. I think this is the case where there are both first and lastprivate clauses specified.

Or do you mean selectiveley enabling privatization for the parallel construct?





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