-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#2068: disable vtAssert test for Intel Classic #2112
Conversation
62ddaa4
to
390f477
Compare
Pipelines resultsPR tests (gcc-10, ubuntu, openmpi, no LB) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (gcc-9, ubuntu, mpich, zoltan) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (gcc-8, ubuntu, mpich, address sanitizer) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (clang-9, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (gcc-7, ubuntu, mpich, trace runtime, LB) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (nvidia cuda 11.0, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (clang-12, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (clang-11, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (gcc-11, ubuntu, mpich, json schema test) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (gcc-12, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (clang-13, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (clang-14, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (nvidia cuda 11.2, ubuntu, mpich) Build for ( UTC)
PR tests (clang-10, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
PR tests (intel icpc, ubuntu, mpich) Build for b0a44b7 (2023-03-22 17:31:22 UTC)
|
@@ -51,7 +51,7 @@ namespace vt { namespace tests { namespace unit { | |||
|
|||
struct TestPreconfig : TestHarness { }; | |||
|
|||
#if not vt_check_enabled(production_build) | |||
#if not vt_check_enabled(production_build) and not defined(__INTEL_COMPILER) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Following Kokkos practice, it's maybe preferable to conditionalize inside the test, and call GTEST_SKIP
in the case where the test won't be run.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2023-03-22T18:05:03.4641692Z Start 387: vt:TestPreconfig.test_vt_assert_no_mpi
2023-03-22T18:05:03.4745283Z 387/487 Test #387: vt:TestPreconfig.test_vt_assert_no_mpi .............................................***Skipped 0.01 sec
Test segfaults when built with icpc 2021.8.0.
390f477
to
b0a44b7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
fixes #2068