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

[via flang-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-flang-fir-hlfir

Author: Kareem Ergawy (ergawy)

<details>
<summary>Changes</summary>

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

---
Full diff: https://github.com/llvm/llvm-project/pull/154303.diff


1 Files Affected:

- (modified) flang/lib/Lower/Bridge.cpp (+1-1) 


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

``````````

</details>


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