[flang-commits] [flang] [mlir] [WIP] Delayed privatization. (PR #79862)

Kareem Ergawy via flang-commits flang-commits at lists.llvm.org
Fri Feb 2 01:41:29 PST 2024


================
@@ -161,6 +161,11 @@ class DataSharingProcessor {
   const Fortran::parser::OmpClauseList &opClauseList;
   Fortran::lower::pft::Evaluation &eval;
 
+  bool useDelayedPrivatizationWhenPossible;
----------------
ergawy wrote:

My initial idea was to **gradually** rollout support for delayed privatization so we would have a mix of eager and delayed privatization based on:
1. The type of the symbol being privatized
2. The operation supporting the `private` clause.
(that exclusion of unsupported symbol types/ops is not implemented yet, hence the test failures in the PR)

That's why I added then `WhenPossible` suffix (maybe this is not the best choice of words here). My aim is that eventually, we can get rid of eager privatization altogether and then there would be any need for this flag.

This is just a way to reduce the scope of the PR and not introduce a humongous change at once.

I hope that makes sense! Please let makes sense. Let me know if you disagree or have better suggestions.

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


More information about the flang-commits mailing list