================
@@ -2029,9 +2031,33 @@ class FirConverter : public 
Fortran::lower::AbstractConverter {
   void handleLocalitySpecs(const IncrementLoopInfo &info) {
     Fortran::semantics::SemanticsContext &semanticsContext =
         bridge.getSemanticsContext();
-    for (const Fortran::semantics::Symbol *sym : info.localSymList)
+    Fortran::lower::omp::DataSharingProcessor dsp(
----------------
ergawy wrote:

Thanks for the review. Would it be ok to add a todo and do this is a separate 
PR? Just to keep the scope of the PR a bit more well defined? 

https://github.com/llvm/llvm-project/pull/138534
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Valentin Clement バレンタイン クレメン via llvm-branch-commits
  • [... Valentin Clement バレンタイン クレメン via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Kareem Ergawy via llvm-branch-commits
  • [... Tom Eccles via llvm-branch-commits

Reply via email to