Skip to content

Commit ec43120

Browse files
[Fortran] Disable gfortran volatile8 test (#239)
This test expects the compiler to give a compilation error. Current LLVM flang does emit an error, but it's a TODO message because volatile variables are not handled yet. There are upcoming PRs to enable this though, and the front end ought to emit some diagnostic about this. Eugene E and Peter K are discussing whether these should be warnings or errors. While this is being discussed, disable the test so work on volatile can proceed. llvm-project#137369
1 parent 0c27673 commit ec43120

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

Fortran/gfortran/regression/DisabledFiles.cmake

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1053,6 +1053,11 @@ file(GLOB FAILING_FILES CONFIGURE_DEPENDS
10531053
binding_label_tests_26b.f90
10541054
test_common_binding_labels_2_main.f03
10551055
string_1.f90 # Expect error on 32 bits platform
1056+
volatile8.f90 # Gfortran expects compilation errors for invalid uses of volatile; flang
1057+
# supports one of these as an extension, and the others ought to either be warnings
1058+
# or errors. See the flang extensions document: "A non-definable actual argument,
1059+
# including the case of a vector subscript, may be associated with an ASYNCHRONOUS or VOLATILE dummy argument"
1060+
# llvm-project#137369
10561061

10571062
# Tests that exercise gfortran's ability to set -std=f95 and then see errors on newer features
10581063
abstract_type_1.f90

0 commit comments

Comments
 (0)