Skip to content

Commit 83658dd

Browse files
authored
[flang][OpenMP] Enable delayed privatization by default for omp.distribute (llvm#131574)
Switches delayed privatization for `omp.distribute` to be on by default: controlled by the `-openmp-enable-delayed-privatization` instead of by `-openmp-enable-delayed-privatization-staging`. ### GFortran & Fujitsu test suite results: #### gfotran test-suite (this PR): ``` Testing Time: 34.51s Passed: 6569 ``` #### Fujitsu without changes (commit: 0813c5c): ``` Testing Time: 155.39s Passed : 88325 Failed : 156 Executable Missing: 408 ``` #### Fujitsu with changes (this PR): ``` Testing Time: 158.54s Passed : 88325 Failed : 156 Executable Missing: 408 ```
1 parent 84e44ae commit 83658dd

File tree

4 files changed

+6
-6
lines changed

4 files changed

+6
-6
lines changed

flang/lib/Lower/OpenMP/OpenMP.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2549,7 +2549,7 @@ static void genStandaloneDistribute(lower::AbstractConverter &converter,
25492549

25502550
DataSharingProcessor dsp(converter, semaCtx, item->clauses, eval,
25512551
/*shouldCollectPreDeterminedSymbols=*/true,
2552-
enableDelayedPrivatizationStaging, symTable);
2552+
enableDelayedPrivatization, symTable);
25532553
dsp.processStep1(&distributeClauseOps);
25542554

25552555
mlir::omp::LoopNestOperands loopNestClauseOps;

flang/test/Lower/OpenMP/distribute.f90

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ subroutine distribute_simple()
77
! CHECK: omp.teams
88
!$omp teams
99

10-
! CHECK: omp.distribute {
10+
! CHECK: omp.distribute private({{.*}}) {
1111
!$omp distribute
1212

1313
! CHECK-NEXT: omp.loop_nest

flang/test/Lower/OpenMP/order-clause.f90

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,15 +61,15 @@ end subroutine do_simd_order_parallel
6161

6262

6363
subroutine distribute_order
64-
!CHECK: omp.distribute order(reproducible:concurrent) {
64+
!CHECK: omp.distribute order(reproducible:concurrent) private({{.*}}) {
6565
!$omp teams distribute order(concurrent)
6666
do i=1,10
6767
end do
68-
!CHECK: omp.distribute order(reproducible:concurrent) {
68+
!CHECK: omp.distribute order(reproducible:concurrent) private({{.*}}) {
6969
!$omp teams distribute order(reproducible:concurrent)
7070
do i=1,10
7171
end do
72-
!CHECK: omp.distribute order(unconstrained:concurrent) {
72+
!CHECK: omp.distribute order(unconstrained:concurrent) private({{.*}}) {
7373
!$omp teams distribute order(unconstrained:concurrent)
7474
do i = 1, 10
7575
end do

flang/test/Transforms/stack-arrays-hlfir.f90

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ end subroutine omp_target_wsloop
7373
! CHECK-NOT: fir.freemem
7474
! CHECK: omp.teams {
7575
! CHECK: fir.alloca !fir.array<2xi64>
76-
! CHECK: omp.distribute {
76+
! CHECK: omp.distribute private({{.*}}) {
7777
! CHECK: omp.loop_nest {{.*}} {
7878
! CHECK-NOT: fir.allocmem
7979
! CHECK-NOT: fir.freemem

0 commit comments

Comments
 (0)