[flang-commits] [flang] [flang] `do concurrent`: Enable delayed localization by default (PR #154303)
Kareem Ergawy via flang-commits
flang-commits at lists.llvm.org
Tue Aug 19 02:47:36 PDT 2025
https://github.com/ergawy updated https://github.com/llvm/llvm-project/pull/154303
>From 1fdedddf1d9e8f91bf4256756cda6203eb2515c4 Mon Sep 17 00:00:00 2001
From: ergawy <kareem.ergawy at amd.com>
Date: Tue, 19 Aug 2025 04:40:34 -0500
Subject: [PATCH] [flang] `do concurrent`: Enable delayed localization by
default
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
---
flang/lib/Lower/Bridge.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
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