This is an archive of the discontinued LLVM Phabricator instance.

Differential D101078

[flang] Update recently added OpenMP tests to use the new driver
ClosedPublic

Authored by awarzynski on Apr 22 2021, 9:19 AM.

Details

Reviewers
sscalpone
jdoerfert
I_Perry
kiranchandramohan
Commits
rG43831d6279ab: [flang] Update recently added OpenMP tests to use the new driver
Summary

Switching from %f18 to %flang_fc1 in LIT tests added in
https://reviews.llvm.org/D91159. This way these tests are run with the
new driver, flang-new, when enabled (i.e. when
FLANG_BUILD_NEW_DRIVER is set).

Diff Detail

Event Timeline

awarzynski created this revision.Apr 22 2021, 9:19 AM
Herald added a reviewer: sscalpone. · View Herald TranscriptApr 22 2021, 9:19 AM
Herald added subscribers: guansong, yaxunl. · View Herald Transcript
awarzynski requested review of this revision.Apr 22 2021, 9:19 AM
Herald added a reviewer: jdoerfert. · View Herald TranscriptApr 22 2021, 9:19 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: llvm-commits, sstefan1. · View Herald Transcript
awarzynski added reviewers: I_Perry, kiranchandramohan.Apr 22 2021, 9:19 AM
awarzynski added a project: Restricted Project.
kiranchandramohan accepted this revision.Apr 22 2021, 9:22 AM

LGTM.

This revision is now accepted and ready to land.Apr 22 2021, 9:22 AM
This revision was landed with ongoing or failed builds.Apr 22 2021, 10:24 AM
Closed by commit rG43831d6279ab: [flang] Update recently added OpenMP tests to use the new driver (authored by awarzynski). · Explain Why
This revision was automatically updated to reflect the committed changes.
awarzynski added a commit: rG43831d6279ab: [flang] Update recently added OpenMP tests to use the new driver.
Harbormaster completed remote builds in B100304: Diff 339676.Apr 22 2021, 11:29 AM

Revision Contents

PathSize
flang/
test/
Semantics/
2 lines
2 lines
2 lines
2 lines
2 lines

Diff 339706

flang/test/Semantics/omp-allocate01.f90

! RUN: %S/test_errors.sh %s %t %f18 -fopenmp ! RUN: %S/test_errors.sh %s %t %flang_fc1 -fopenmp
! OpenMP Version 5.0 ! OpenMP Version 5.0
! 2.11.3 allocate Directive ! 2.11.3 allocate Directive
! The allocate directive must appear in the same scope as the declarations of ! The allocate directive must appear in the same scope as the declarations of
! each of its list items and must follow all such declarations. ! each of its list items and must follow all such declarations.
subroutine allocate() subroutine allocate()
use omp_lib use omp_lib
integer :: x integer :: x
Show All 15 Lines

flang/test/Semantics/omp-allocate02.f90

! RUN: %S/test_errors.sh %s %t %f18 -fopenmp ! RUN: %S/test_errors.sh %s %t %flang_fc1 -fopenmp
! OpenMP Version 5.0 ! OpenMP Version 5.0
! 2.11.3 allocate Directive ! 2.11.3 allocate Directive
! At most one allocator clause can appear on the allocate directive. ! At most one allocator clause can appear on the allocate directive.
subroutine allocate() subroutine allocate()
use omp_lib use omp_lib
integer :: x, y integer :: x, y
integer :: a, b integer :: a, b
Show All 15 Lines

flang/test/Semantics/omp-allocate03.f90

! RUN: %S/test_errors.sh %s %t %f18 -fopenmp ! RUN: %S/test_errors.sh %s %t %flang_fc1 -fopenmp
! OpenMP Version 5.0 ! OpenMP Version 5.0
! 2.11.3 allocate Directive ! 2.11.3 allocate Directive
! A variable that is part of another variable (as an array or ! A variable that is part of another variable (as an array or
! structure element) cannot appear in an allocate directive. ! structure element) cannot appear in an allocate directive.
subroutine allocate() subroutine allocate()
use omp_lib use omp_lib
type my_type type my_type
Show All 15 Lines

flang/test/Semantics/omp-allocate04.f90

! RUN: %S/test_errors.sh %s %t %f18 -fopenmp ! RUN: %S/test_errors.sh %s %t %flang_fc1 -fopenmp
! OpenMP Version 5.0 ! OpenMP Version 5.0
! 2.11.3 allocate Directive ! 2.11.3 allocate Directive
! Only the allocator clause is allowed on the allocate directive ! Only the allocator clause is allowed on the allocate directive
subroutine allocate() subroutine allocate()
use omp_lib use omp_lib
integer :: x, y integer :: x, y
!$omp allocate(x) allocator(omp_default_mem_alloc) !$omp allocate(x) allocator(omp_default_mem_alloc)
!ERROR: PRIVATE clause is not allowed on the ALLOCATE directive !ERROR: PRIVATE clause is not allowed on the ALLOCATE directive
!$omp allocate(y) private(y) !$omp allocate(y) private(y)
end subroutine allocate end subroutine allocate

flang/test/Semantics/omp-allocate05.f90

! RUN: %S/test_errors.sh %s %t %f18 -fopenmp ! RUN: %S/test_errors.sh %s %t %flang_fc1 -fopenmp
! OpenMP Version 5.0 ! OpenMP Version 5.0
! 2.11.3 allocate Directive ! 2.11.3 allocate Directive
! allocate directives that appear in a target region must specify an allocator ! allocate directives that appear in a target region must specify an allocator
! clause unless a requires directive with the dynamic_allocators clause is present ! clause unless a requires directive with the dynamic_allocators clause is present
! in the same compilation unit. ! in the same compilation unit.
subroutine allocate() subroutine allocate()
use omp_lib use omp_lib
Show All 15 Lines