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

[Kareem Ergawy via flang-commits]

================
@@ -487,15 +495,54 @@ void DataSharingProcessor::collectDefaultSymbols() {
   }
 }
 
-void DataSharingProcessor::privatize() {
+void DataSharingProcessor::privatize(
+    llvm::SetVector<mlir::omp::PrivateClauseOp> *privateInitializers) {
----------------
ergawy wrote:

Thanks for taking a look!

Just to be sure we are on the same page, are you referring to using the push/pop scope mechanism instead of the `privatizerSymbolMap` I create in line 529 below?

The `privateInitializers` argument is not related to scoping and symbol map management. Its purpose is to be populated with the created ops so that we can use them as operands to the `omp.parallel`/`omp.target` ops (or any op with a `private` clause.

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