[flang-commits] [flang] 0e93dbc - [flang] `do concurrent`: Enable delayed localization by default (#154303)

via flang-commits flang-commits at lists.llvm.org
Tue Aug 19 03:07:20 PDT 2025


Author: Kareem Ergawy
Date: 2025-08-19T12:07:17+02:00
New Revision: 0e93dbc6b1cac9c69c546cff7b5dd5935917ae9e

URL: https://github.com/llvm/llvm-project/commit/0e93dbc6b1cac9c69c546cff7b5dd5935917ae9e
DIFF: https://github.com/llvm/llvm-project/commit/0e93dbc6b1cac9c69c546cff7b5dd5935917ae9e.diff

LOG: [flang] `do concurrent`: Enable delayed localization by default (#154303)

Enables delayed localization by default for `do concurrent`. Tested both
gfortran and Fujitsu test suites.

All tests pass for gfortran tests. For Fujitsu, enabled delayed
localization passes more tests:

Delayed localization disabled:
Testing Time: 7251.76s
  Passed            : 88520
  Failed            :   162
  Executable Missing:   408

Delayed localization enabled:
Testing Time: 7216.73s
  Passed            : 88522
  Failed            :   160
  Executable Missing:   408

Added: 
    

Modified: 
    flang/lib/Lower/Bridge.cpp

Removed: 
    


################################################################################
diff  --git a/flang/lib/Lower/Bridge.cpp b/flang/lib/Lower/Bridge.cpp
index ab7bf28a9e8bf..ea98aa2804944 100644
--- a/flang/lib/Lower/Bridge.cpp
+++ b/flang/lib/Lower/Bridge.cpp
@@ -2064,8 +2064,7 @@ class FirConverter : public Fortran::lower::AbstractConverter {
     // TODO Promote to using `enableDelayedPrivatization` (which is enabled by
     // default unlike the staging flag) once the implementation of this is more
     // complete.
-    bool useDelayedPriv =
-        enableDelayedPrivatizationStaging && doConcurrentLoopOp;
+    bool useDelayedPriv = enableDelayedPrivatization && doConcurrentLoopOp;
     llvm::SetVector<const Fortran::semantics::Symbol *> allPrivatizedSymbols;
     llvm::SmallSet<const Fortran::semantics::Symbol *, 16> mightHaveReadHostSym;
 


        


More information about the flang-commits mailing list