Skip to content

Commit

Permalink
[test] Add check-flang-long target for running NCAR tests via make
Browse files Browse the repository at this point in the history
Also update the GitHub workflows to run `check-flang-long` instead of
`check-all`. The latter remains synonymous to `check-flang`, which
skips the NCAR kernel tests as UNSUPPORTED.
  • Loading branch information
bryanpkc committed Jun 21, 2023
1 parent 84541c6 commit a3bd452
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build_flang.yml
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ jobs:
- name: Test flang
run: |
cd build/flang
make check-all
make check-flang-long
# Archive documentation just once, for the fastest job.
- if: matrix.cc == 'clang' && matrix.version == '11'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/build_flang_arm64.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,4 +64,4 @@ jobs:
- name: Test flang
run: |
cd ${{ env.build_path }}/flang/build/flang
make check-all
make check-flang-long
9 changes: 9 additions & 0 deletions test/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,15 @@ add_lit_testsuite(check-flang "Running Flang regression tests"
)
set_target_properties(check-flang PROPERTIES FOLDER "Flang tests")

add_lit_testsuite(check-flang-long "Running time-consuming Flang regression tests"
${CMAKE_CURRENT_BINARY_DIR}
EXCLUDE_FROM_CHECK_ALL
PARAMS "${FLANG_TEST_PARAMS};run_long_tests=true"
DEPENDS ${FLANG_TEST_DEPS}
ARGS ${FLANG_TEST_EXTRA_ARGS}
)
set_target_properties(check-flang-long PROPERTIES FOLDER "Flang tests")

add_lit_testsuites(FLANG ${CMAKE_CURRENT_SOURCE_DIR}
PARAMS ${FLANG_TEST_PARAMS}
DEPENDS ${FLANG_TEST_DEPS}
Expand Down

0 comments on commit a3bd452

Please sign in to comment.