Remove unnecessary compiler behavior test in modules test suite

The “preprocessor output” test in the modules test suite only tests
compiler behavior, and the string it looks for in the preprocessed
output is different for older and newer Clang versions. It seems like a
kind of a probing call, but I don’t see why we would want to probe the
compiler behavior in this case so I’ll just remove it.

Fixes #601.

diff --git a/test/suites/modules.bash b/test/suites/modules.bash
index dcd02abd665d46a011edeee8781bae923f46714f..35dc04c24c5807b11f3aaf013f4558bade514059 100644 (file)
--- a/test/suites/modules.bash
+++ b/test/suites/modules.bash
@@ -3,7 +3,7 @@ SUITE_modules_PROBE() {
         echo "-fmodules/-fcxx-modules not supported by compiler"
         return
     fi
- }
+}
 
 SUITE_modules_SETUP() {
     unset CCACHE_NODIRECT
@@ -29,11 +29,6 @@ EOF
 }
 
 SUITE_modules() {
-    # -------------------------------------------------------------------------
-    TEST "preprocessor output"
-    $COMPILER -fmodules -fcxx-modules test1.cpp -E > test1.preprocessed.cpp
-    expect_file_contains "test1.preprocessed.cpp" "#pragma clang module import Test1"
-
     # -------------------------------------------------------------------------
     TEST "fall back to real compiler, no sloppiness"