[Fortran/gfortran] Remove HLFIR-specific allow and deny lists

The code that supports "feature-specific" testing has been retained, but has
been tweaked a little bit. The README has been updated with some examples.

Author: tarunprabhu

Fortran/gfortran/CMakeLists.txt

@@ -473,28 +473,25 @@ function(gfortran_populate_disabled_tests out)
   list(APPEND skipped ${SKIPPED_FILES})
   list(APPEND failing ${FAILING_FILES})
 
-  # do the same for any requested feature extentions
+  # do the same for any requested feature extensions
   foreach(feature ${TEST_SUITE_FORTRAN_FEATURES})
     set(UNSUPPORTED_FILES "")
     set(UNIMPLEMENTED_FILES "")
     set(SKIPPED_FILES "")
     set(FAILING_FILES "")
-    include(${CMAKE_CURRENT_SOURCE_DIR}/DisabledFiles${feature}.cmake)
+    include(${CMAKE_CURRENT_SOURCE_DIR}/DisabledFiles_${feature}.cmake)
     list(APPEND unsupported ${UNSUPPORTED_FILES})
     list(APPEND unimplemented ${UNIMPLEMENTED_FILES})
     list(APPEND skipped ${SKIPPED_FILES})
     list(APPEND failing ${FAILING_FILES})
 
     # enable any tests that now pass for this feature
-    set(SUPPORTED_FILES "")
-    set(IMPLEMENTED_FILES "")
-    set(UNSKIPPED_FILES "")
-    set(PASSING_FILES "")
-    include(${CMAKE_CURRENT_SOURCE_DIR}/EnabledFiles${feature}.cmake)
-    list(REMOVE_ITEM unsupported ${SUPPORTED_FILES})
-    list(REMOVE_ITEM unimplemented ${IMPLEMENTED_FILES})
-    list(REMOVE_ITEM skipped ${UNSKIPPED_FILES})
-    list(REMOVE_ITEM failing ${PASSING_FILES})
+    set(ENABLED_FILES "")
+    include(${CMAKE_CURRENT_SOURCE_DIR}/EnabledFiles_${feature}.cmake)
+    list(REMOVE_ITEM unsupported ${ENABLED_FILES})
+    list(REMOVE_ITEM unimplemented ${ENABLED_FILES})
+    list(REMOVE_ITEM skipped ${ENABLED_FILES})
+    list(REMOVE_ITEM failing ${ENABLED_FILES})
   endforeach()
 
   set(disabled "")
@@ -511,7 +508,7 @@ function(gfortran_populate_disabled_tests out)
   # and unimplemented tests since some could be enabled once some feature is
   # implemented. Eventually, all the TEST_SUITE_FORTRAN_FORCE_* options (perhaps
   # with the exception of TEST_SUITE_FORTRAN_FORCE_UNSUPPORTED_TESTS) should
-  # become redundant and removed.
+  # become redundant and should be removed.
   if (NOT TEST_SUITE_FORTRAN_FORCE_ALL_TESTS AND
       NOT TEST_SUITE_FORTRAN_FORCE_UNIMPLEMENTED_TESTS)
     list(APPEND disabled ${unimplemented})

Fortran/gfortran/README.md

@@ -115,14 +115,35 @@ are unrelated to the gfortran tests here.
 ### Testing non-standard features/flags
 
 Additional denylists for a particular feature can be included by creating
-DisabledFilesFEATURE.cmake files (in the same format as those for the default
-denylists), and adding FEATURE to `TEST_SUITE_FORTRAN_FEATURES`. Additional
-compiler flags can be added using `CMAKE_Fortran_FLAGS`.
+`DisabledFiles_FEATURE.cmake` files (in the same format as those for the default
+denylists, `DisabledFiles.cmake`), and adding FEATURE to 
+`TEST_SUITE_FORTRAN_FEATURES`. Additional compiler flags can be added using
+`CMAKE_Fortran_FLAGS`.
 
-For example, to test HLFIR one could use
-`CMAKE_Fortran_Flags=-flang-experimental-hlfir` and
-`TEST_SUITE_FORTRAN_FEATURES=HLFIR`.
+For example, to test feature, FOO, one could use
 
+```
+cmake -DTEST_SUITE_FORTRAN_FEATURES=FOO \
+      -DCMAKE_Fortran_FLAGS=-some-foo-specific-flag-if-required \
+      <other cmake flags>
+```
+
+`DisabledFiles_FOO.cmake` files can be created in the appropriate subdirectories
+if enabling the feature/flag results in the failure of tests that otherwise pass.
+Conversely, the feature/flag may cause some disabled tests to pass. These can be
+added to an allowlist file, `EnabledFiles_FOO.cmake` in the corresponding 
+directory. The file must contain a single variable named `ENABLED_FILES` with 
+the file names of the tests that should be enabled (in the case of multi-file
+tests, this should be the name of the "main" file). An example of such a list is
+below.
+
+```
+file(GLOB ENABLED_FILES CONFIGURE_DEPENDS
+  test_1.f90
+  multifile-main.f03
+)
+
+```
 
 ### Notes for developers/maintainers ###