From 7abb6e99df7b1e99b2262af733e6d527c3fdd1c0 Mon Sep 17 00:00:00 2001 From: Luke Ireland Date: Fri, 6 Mar 2020 11:21:36 +0000 Subject: [PATCH] Added CMakeLists changes, moved config and made test scripts compatible. All Fortran tests are now run in lit, except Preprocessing tests #1052 Preprocessing tests are a separate kind of test, so will be sorted out later. --- CMakeLists.txt | 2 +- {test-lit => test}/CMakeLists.txt | 0 test/Evaluate/lit.local.cfg.py | 10 + test/Evaluate/test_folding.sh | 17 +- test/Preprocessing/lit.local.cfg.py | 12 + test/Semantics/CMakeLists.txt | 346 -------------------------- test/Semantics/common.sh | 17 +- test/Semantics/test_any.sh | 4 +- {test-lit => test}/lit.cfg.py | 10 +- {test-lit => test}/lit.site.cfg.py.in | 2 +- unittests/Evaluate/CMakeLists.txt | 8 - 11 files changed, 41 insertions(+), 387 deletions(-) rename {test-lit => test}/CMakeLists.txt (100%) create mode 100644 test/Evaluate/lit.local.cfg.py create mode 100644 test/Preprocessing/lit.local.cfg.py delete mode 100644 test/Semantics/CMakeLists.txt rename {test-lit => test}/lit.cfg.py (90%) rename {test-lit => test}/lit.site.cfg.py.in (92%) diff --git a/CMakeLists.txt b/CMakeLists.txt index dedcdc21f142..957e8daeb719 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -147,7 +147,7 @@ add_subdirectory(lib) add_subdirectory(runtime) add_subdirectory(unittests) add_subdirectory(tools) -add_subdirectory(test-lit) +add_subdirectory(test) configure_file( ${FLANG_SOURCE_DIR}/include/flang/Config/config.h.cmake diff --git a/test-lit/CMakeLists.txt b/test/CMakeLists.txt similarity index 100% rename from test-lit/CMakeLists.txt rename to test/CMakeLists.txt diff --git a/test/Evaluate/lit.local.cfg.py b/test/Evaluate/lit.local.cfg.py new file mode 100644 index 000000000000..68daa63595fa --- /dev/null +++ b/test/Evaluate/lit.local.cfg.py @@ -0,0 +1,10 @@ +# -*- Python -*- + +import os +import platform +import sys + +import lit.formats +from lit.llvm import llvm_config + +config.environment["LIBPGMATH"] = "False" diff --git a/test/Evaluate/test_folding.sh b/test/Evaluate/test_folding.sh index 84b10eb23609..56c1bc9c9e25 100755 --- a/test/Evaluate/test_folding.sh +++ b/test/Evaluate/test_folding.sh @@ -19,26 +19,21 @@ # - test_x is not folded (it is neither .true. nor .false.). This means the # compiler could not fold the expression. -PATH=/usr/bin:/bin -srcdir=$(dirname $0) -F18CC=${F18:-../../../tools/f18/bin/f18} -CMD="$F18CC -fdebug-dump-symbols -fparse-only" +CMD="$2 -fdebug-dump-symbols -fparse-only" if [[ $# < 1 ]]; then - echo "Usage: $0 [-pgmath=]" + echo "Usage: $0 [] " exit 1 fi -src=$srcdir/$1 +src=$1 [[ ! -f $src ]] && echo "File not found: $src" && exit 1 -temp=temp-$1 -rm -rf $temp -mkdir $temp -[[ $KEEP ]] || trap "rm -rf $temp" EXIT +temp=$3 +mkdir -p $temp # Check if tests should assume folding is using libpgmath -if [[ $# > 1 && "$2" = "-pgmath=true" ]]; then +if [[ $LIBPGMATH == "True" ]]; then CMD="$CMD -DTEST_LIBPGMATH" echo "Assuming libpgmath support" fi diff --git a/test/Preprocessing/lit.local.cfg.py b/test/Preprocessing/lit.local.cfg.py new file mode 100644 index 000000000000..9f1669ca4979 --- /dev/null +++ b/test/Preprocessing/lit.local.cfg.py @@ -0,0 +1,12 @@ +# -*- Python -*- + +import os +import platform +import sys + +import lit.formats +from lit.llvm import llvm_config + +# Added this lit config file to prevent lit from discovering these tests +# See Issue #1052 +config.suffixes = [] diff --git a/test/Semantics/CMakeLists.txt b/test/Semantics/CMakeLists.txt deleted file mode 100644 index a24108782ae7..000000000000 --- a/test/Semantics/CMakeLists.txt +++ /dev/null @@ -1,346 +0,0 @@ -#===-- test/Semantics/CMakeLists.txt ---------------------------------------===# -# -# Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. -# See https://llvm.org/LICENSE.txt for license information. -# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception -# -#===------------------------------------------------------------------------===# - -# Run tests with test_errors.sh. It compiles the test with f18 and compares -# actual errors produced with expected ones listed in the source. - -# These test files have expected errors in the source -set(ERROR_TESTS - implicit01.f90 - implicit02.f90 - implicit03.f90 - implicit04.f90 - implicit05.f90 - implicit06.f90 - implicit07.f90 - implicit08.f90 - int-literals.f90 - io01.f90 - io02.f90 - io03.f90 - io04.f90 - io05.f90 - io06.f90 - io07.f90 - io08.f90 - io09.f90 - io10.f90 - kinds02.f90 - kinds04.f90 - resolve01.f90 - resolve02.f90 - resolve03.f90 - resolve04.f90 - resolve05.f90 - resolve06.f90 - resolve07.f90 - resolve08.f90 - resolve09.f90 - resolve10.f90 - resolve11.f90 - resolve12.f90 - resolve13.f90 - resolve14.f90 - resolve15.f90 - resolve16.f90 - resolve17.f90 - resolve18.f90 - resolve19.f90 - resolve20.f90 - resolve21.f90 - resolve22.f90 - resolve23.f90 - resolve24.f90 - resolve25.f90 - resolve26.f90 - resolve27.f90 - resolve28.f90 - resolve29.f90 - resolve30.f90 - resolve31.f90 - resolve32.f90 - resolve33.f90 - resolve34.f90 - resolve35.f90 - resolve36.f90 - resolve37.f90 - resolve38.f90 - resolve39.f90 - resolve40.f90 - resolve41.f90 - resolve42.f90 - resolve43.f90 - resolve44.f90 - resolve45.f90 - resolve46.f90 - resolve47.f90 - resolve48.f90 - resolve49.f90 - resolve50.f90 - resolve51.f90 - resolve52.f90 - resolve53.f90 - resolve54.f90 - resolve55.f90 - resolve56.f90 - resolve57.f90 - resolve58.f90 - resolve59.f90 - resolve60.f90 - resolve61.f90 - resolve62.f90 - resolve63.f90 - resolve64.f90 - resolve65.f90 - resolve66.f90 - resolve67.f90 - resolve68.f90 - resolve69.f90 - resolve70.f90 - resolve71.f90 - resolve72.f90 - resolve73.f90 - resolve74.f90 - resolve75.f90 - stop01.f90 - structconst01.f90 - structconst02.f90 - structconst03.f90 - structconst04.f90 - assign01.f90 - assign02.f90 - assign03.f90 - assign04.f90 - if_arith02.f90 - if_arith03.f90 - if_arith04.f90 - if_construct02.f90 - if_stmt02.f90 - if_stmt03.f90 - computed-goto01.f90 - computed-goto02.f90 - nullify01.f90 - nullify02.f90 - deallocate01.f90 - deallocate04.f90 - deallocate05.f90 - coarrays01.f90 - altreturn01.f90 - altreturn02.f90 - altreturn03.f90 - altreturn04.f90 - altreturn05.f90 - allocate01.f90 - allocate02.f90 - allocate03.f90 - allocate04.f90 - allocate05.f90 - allocate06.f90 - allocate07.f90 - allocate08.f90 - allocate09.f90 - allocate10.f90 - allocate11.f90 - allocate12.f90 - allocate13.f90 - doconcurrent01.f90 - doconcurrent05.f90 - doconcurrent06.f90 - doconcurrent08.f90 - dosemantics01.f90 - dosemantics02.f90 - dosemantics03.f90 - dosemantics04.f90 - dosemantics05.f90 - dosemantics06.f90 - dosemantics07.f90 - dosemantics08.f90 - dosemantics09.f90 - dosemantics10.f90 - dosemantics11.f90 - dosemantics12.f90 - expr-errors01.f90 - expr-errors02.f90 - null01.f90 - omp-resolve01.f90 - omp-resolve02.f90 - omp-resolve03.f90 - omp-resolve04.f90 - omp-resolve05.f90 - omp-clause-validity01.f90 - omp-loop-association.f90 -# omp-nested01.f90 - omp-declarative-directive.f90 - omp-atomic.f90 - omp-device-constructs.f90 - equivalence01.f90 - init01.f90 - if_arith01.f90 - if_construct01.f90 - if_stmt01.f90 - blockconstruct01.f90 - blockconstruct02.f90 - blockconstruct03.f90 - call01.f90 - call02.f90 - call03.f90 - call04.f90 - call05.f90 - call06.f90 - call07.f90 - call08.f90 - call09.f90 - call10.f90 - call11.f90 - call12.f90 - call13.f90 - call14.f90 - call15.f90 - forall01.f90 - misc-declarations.f90 - separate-module-procs.f90 - bindings01.f90 - bad-forward-type.f90 - c_f_pointer.f90 - critical01.f90 - critical02.f90 - critical03.f90 - block-data01.f90 - complex01.f90 - data01.f90 - data02.f90 - namelist01.f90 -) - -# These test files have expected symbols in the source -set(SYMBOL_TESTS - symbol01.f90 - symbol02.f90 - symbol03.f90 - symbol05.f90 - symbol06.f90 - symbol07.f90 - symbol08.f90 - symbol09.f90 - symbol10.f90 - symbol11.f90 - symbol12.f90 - symbol13.f90 - symbol14.f90 - symbol15.f90 - symbol16.f90 - symbol17.f90 - omp-symbol01.f90 - omp-symbol02.f90 - omp-symbol03.f90 - omp-symbol04.f90 - omp-symbol05.f90 - omp-symbol06.f90 - omp-symbol07.f90 - omp-symbol08.f90 - kinds01.f90 - kinds03.f90 - procinterface01.f90 -) - -# These test files have expected .mod file contents in the source -set(MODFILE_TESTS - modfile01.f90 - modfile02.f90 - modfile03.f90 - modfile04.f90 - modfile05.f90 - modfile06.f90 - modfile07.f90 - modfile08.f90 - modfile09-*.f90 - modfile10.f90 - modfile11.f90 - modfile12.f90 - modfile13.f90 - modfile14.f90 - modfile15.f90 - modfile16.f90 - modfile17.f90 - modfile18.f90 - modfile19.f90 - modfile20.f90 - modfile21.f90 - modfile22.f90 - modfile23.f90 - modfile24.f90 - modfile25.f90 - modfile26.f90 - modfile27.f90 - modfile28.f90 - modfile29.f90 - modfile30.f90 - modfile31.f90 - modfile32.f90 - modfile33.f90 - modfile34.f90 - modfile35.f90 -) - -set(LABEL_TESTS - label*.[Ff]90 -) - -set(DOCONCURRENT_TESTS - doconcurrent02.f90 - doconcurrent03.f90 - doconcurrent04.f90 - doconcurrent07.f90 -) - -set(CANONDO_TESTS - canondo*.[Ff]90 -) - -set(CRITICAL_TESTS - critical04.f90 -) - -set(GETSYMBOLS_TESTS - getsymbols01.f90 - getsymbols02-*.f90 - getsymbols03-a.f90 - getsymbols04.f90 - getsymbols05.f90 -) - -set(GETDEFINITION_TESTS - getdefinition01.f90 - getdefinition02.f - getdefinition03-a.f90 - getdefinition04.f90 - getdefinition05.f90 -) - -set(F18 $) - -foreach(test ${ERROR_TESTS}) - add_test(NAME ${test} - COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/test_errors.sh ${test} ${F18}) -endforeach() - -foreach(test ${SYMBOL_TESTS}) - add_test(NAME ${test} - COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/test_symbols.sh ${test} ${F18}) -endforeach() - -foreach(test ${MODFILE_TESTS}) - add_test(NAME ${test} - COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/test_modfile.sh ${test} ${F18}) -endforeach() - -foreach(test ${LABEL_TESTS} ${CANONDO_TESTS} ${DOCONCURRENT_TESTS} - ${CRITICAL_TESTS} ${GETSYMBOLS_TESTS} ${GETDEFINITION_TESTS}) - add_test(NAME ${test} - COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/test_any.sh ${test} ${F18}) -endforeach() diff --git a/test/Semantics/common.sh b/test/Semantics/common.sh index e84d9b933631..9924110f0ee0 100644 --- a/test/Semantics/common.sh +++ b/test/Semantics/common.sh @@ -10,19 +10,18 @@ function die { echo "$(basename $0): $*" >&2 exit 1 } +if [[ $# < 1 ]]; then + echo "Usage: $(basename $0) [] " + exit 1 +fi -case $# in - (1) ;; - (2) F18=$2 ;; - (*) echo "Usage: $(basename $0) []"; exit 1 -esac -[[ -z ${F18+x} ]] && die "Path to f18 must be second argument or in F18 environment variable" -[[ ! -f $F18 ]] && die "f18 executable not found: $F18" case $1 in (/*) src=$1 ;; (*) src=$(dirname $0)/$1 ;; esac USER_OPTIONS=`sed -n 's/^ *! *OPTIONS: *//p' $src` echo $USER_OPTIONS -temp=`mktemp -d ./tmp.XXXXXX` -[[ $KEEP ]] || trap "rm -rf $temp" EXIT +F18=$2 +[[ ! -f $F18 ]] && die "f18 executable not found: $F18" +temp=$3 +mkdir -p $temp diff --git a/test/Semantics/test_any.sh b/test/Semantics/test_any.sh index 71f42889911a..b0735935928f 100755 --- a/test/Semantics/test_any.sh +++ b/test/Semantics/test_any.sh @@ -48,9 +48,9 @@ function internal_check() { } gr=0 -for input in ${srcdir}/$*; do +for input in $1; do [[ ! -f $input ]] && die "File not found: $input" - CMD=$(cat ${input} | egrep '^[[:space:]]*![[:space:]]*RUN:[[:space:]]*' | sed -e 's/^[[:space:]]*![[:space:]]*RUN:[[:space:]]*//') + CMD=$(cat ${input} | egrep '^[[:space:]]*![[:space:]]*EXEC:[[:space:]]*' | sed -e 's/^[[:space:]]*![[:space:]]*EXEC:[[:space:]]*//') CMD=$(echo ${CMD} | sed -e "s:%s:${input}:g") if egrep -q -e '%t' <<< ${CMD} ; then CMD=$(echo ${CMD} | sed -e "s:%t:$temp/t:g") diff --git a/test-lit/lit.cfg.py b/test/lit.cfg.py similarity index 90% rename from test-lit/lit.cfg.py rename to test/lit.cfg.py index 3ca3c8ad23f7..439af557b1d4 100644 --- a/test-lit/lit.cfg.py +++ b/test/lit.cfg.py @@ -24,19 +24,11 @@ # the test runner updated. config.test_format = lit.formats.ShTest(not llvm_config.use_lit_shell) - # suffixes: A list of file extensions to treat as test files. config.suffixes = ['.f', '.F', '.ff','.FOR', '.for', '.f77', '.f90', '.F90', '.ff90', '.f95', '.F95', '.ff95', '.fpp', '.FPP', '.cuf', '.CUF', '.f18', '.F18'] -# test_source_root: The root path where tests are located. -config.test_source_root = os.path.dirname(__file__) - - -# test_exec_root: The root path where tests should be run. -config.test_exec_root = os.path.join(config.flang_obj_root, 'test-lit') - config.substitutions.append(('%PATH%', config.environment['PATH'])) llvm_config.use_default_substitutions() @@ -50,7 +42,7 @@ config.test_source_root = os.path.dirname(__file__) # test_exec_root: The root path where tests should be run. -config.test_exec_root = os.path.join(config.flang_obj_root, 'test-lit') +config.test_exec_root = os.path.join(config.flang_obj_root, 'test') # Tweak the PATH to include the tools dir. llvm_config.with_environment('PATH', config.flang_tools_dir, append_path=True) diff --git a/test-lit/lit.site.cfg.py.in b/test/lit.site.cfg.py.in similarity index 92% rename from test-lit/lit.site.cfg.py.in rename to test/lit.site.cfg.py.in index d00f3856fb41..34bcdab11b55 100644 --- a/test-lit/lit.site.cfg.py.in +++ b/test/lit.site.cfg.py.in @@ -23,4 +23,4 @@ import lit.llvm lit.llvm.initialize(lit_config, config) # Let the main config do the real work. -lit_config.load_config(config, "@FLANG_SOURCE_DIR@/test-lit/lit.cfg.py") +lit_config.load_config(config, "@FLANG_SOURCE_DIR@/test/lit.cfg.py") diff --git a/unittests/Evaluate/CMakeLists.txt b/unittests/Evaluate/CMakeLists.txt index e5a1d30cd0d3..d874fcb39dbc 100644 --- a/unittests/Evaluate/CMakeLists.txt +++ b/unittests/Evaluate/CMakeLists.txt @@ -140,11 +140,3 @@ add_test(Real real-test) add_test(RESHAPE reshape-test) add_test(ISO-binding ISO-Fortran-binding-test) add_test(folding folding-test) - -set(TEST_LIBPGMATH "-pgmath=false") -if (LIBPGMATH_DIR) - find_library(LIBPGMATH pgmath PATHS ${LIBPGMATH_DIR}) - if(LIBPGMATH) - set(TEST_LIBPGMATH "-pgmath=true") - endif() -endif()