Skip to content

Commit cb81ffd

Browse files
committed
[Fortran] Remove references to -flang-experimental-exec
Remove all references to -flang-experimental-exec, both in the README's and the build files since it has been removed from flang. Differential Revision: https://reviews.llvm.org/D153551
1 parent 98c11fe commit cb81ffd

File tree

4 files changed

+0
-20
lines changed

4 files changed

+0
-20
lines changed

Fortran/UnitTests/finalization/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ a command similar to the following can be used from within test-suite-build:
5959
``` shell
6060
cmake -DCMAKE_BUILD_TYPE=Release \
6161
-DCMAKE_Fortran_COMPILER:FILEPATH=/home/users/<you>/llvm-project/build/bin/flang-new \
62-
-DCMAKE_Fortran_FLAGS=-flang-experimental-exec \
6362
-DTEST_SUITE_FORTRAN:BOOL=On \
6463
-DTEST_SUITE_SUBDIRS=Fortran/UnitTests/finalization \
6564
../test-suite

Fortran/gfortran/CMakeLists.txt

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -6,15 +6,6 @@ include(CheckFortranCompilerFlag)
66
# support for the flag is added, this should be removed.
77
remove_definitions(-w)
88

9-
# If the test suite was configured by adding -flang-experimental-exec, remove
10-
# that flag from the default compilation flags. It is only needed at link-time
11-
# anyway, and for that it is explicitly added to those targets that need it.
12-
#
13-
# NOTE: This should be removed once the -flang-experimental-exec flag is no
14-
# longer needed.
15-
string(REPLACE "-flang-experimental-exec" "" TMP_FFLAGS "${CMAKE_Fortran_FLAGS}")
16-
set(CMAKE_Fortran_FLAGS "${TMP_FFLAGS}")
17-
189
# This option is added because as of 2023/03/20, several tests in this
1910
# directory have been disabled. Some of them exercise unsupported non-standard
2011
# extensions, others trigger a "not yet implemented" assertion while some cause
@@ -310,9 +301,6 @@ function(gfortran_execute_test File)
310301
endif()
311302
endforeach()
312303

313-
# NOTE: This should be removed when the -flang-experimental-exec flag is no
314-
# longer needed.
315-
target_link_options(${test_target} PRIVATE -flang-experimental-exec)
316304
set_target_properties(${test_target} PROPERTIES LINKER_LANGUAGE Fortran)
317305
endfunction()
318306

Fortran/gfortran/README.md

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,6 @@ fail to build for the reasons described above. The *failing* tests do build.
5959

6060
### Usage ###
6161

62-
As of 18-Apr-2023, the `-flang-experimental-exec` flag may need to be used when
63-
configuring and compiling these tests. This can be passed to `cmake` via
64-
`CMAKE_Fortran_FLAGS`.
65-
6662
By default, the *unsupported*, *unimplemented*, *skipped*, and *failing* tests
6763
are not run. The intention is that all tests in the test suite should pass by
6864
default.
@@ -91,7 +87,6 @@ cmake -DCMAKE_BUILD_TYPE=Release \
9187
-DCMAKE_C_COMPILER=/path/to/clang \
9288
-DCMAKE_CXX_COMPILER=/path/to/clang++ \
9389
-DCMAKE_Fortran_COMPILER=/path/to/flang-new \
94-
-DCMAKE_Fortran_FLAGS=-flang-experimental-exec \
9590
-DTEST_SUITE_FORTRAN=On \
9691
-DTEST_SUITE_SUBDIRS=Fortran \
9792
-DTEST_SUITE_FORTRAN_ISO_C_HEADER_DIR=/path/to/dir/containing/header \

Fortran/gfortran/torture/compile/CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,6 @@ function(add_tests UnsupportedTests UnimplementedTests SkippedTests FailingTests
9090

9191
file(WRITE ${DummySrc} "program test\nend program test")
9292
add_executable(${Dummy} ${DummySrc})
93-
# At some point, the -flang-experimental-exec flag will be removed.
94-
target_link_options(${Dummy} PUBLIC "-flang-experimental-exec")
9593

9694
# All the "compile" tests in the gfortran torture tests are expected to
9795
# pass. Since diagnostics are only saved on failure, the diagnostics

0 commit comments

Comments
 (0)