Skip to content

Commit

Permalink
Allow the disable of just the LAPACK test for STEQR() (trilinos#2410)
Browse files Browse the repository at this point in the history
This is the only function that continues to segfault on Power8 and Power9
builds using NETLIB LAPACK in debug builds.  All of the Trilinos code
downstream has removed a dependence on this LAPACK function. This will allow
the rest of the LAPACK tests to be run in Power8 and Power9 'debug' builds.
  • Loading branch information
bartlettroscoe committed Dec 17, 2018
1 parent f94ba63 commit f8e692d
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 1 deletion.
7 changes: 6 additions & 1 deletion packages/teuchos/numerics/test/LAPACK/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,8 +1,13 @@

IF (${PACKAGE_NAME}_DISABLE_STEQR_TEST)
SET(${PACKAGE_NAME}_LAPACK_test_DEFINES -DTEUCHOSNUMERICS_DISABLE_STEQR_TEST)
ELSE()
SET(${PACKAGE_NAME}_LAPACK_test_DEFINES)
ENDIF()

TRIBITS_ADD_EXECUTABLE_AND_TEST(
LAPACK_test
SOURCES cxx_main.cpp
TARGET_DEFINES ${${PACKAGE_NAME}_LAPACK_test_DEFINES}
ARGS -v
COMM serial mpi
NUM_MPI_PROCS 1
Expand Down
8 changes: 8 additions & 0 deletions packages/teuchos/numerics/test/LAPACK/cxx_main.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,8 @@ int main(int argc, char* argv[])

if (verbose) std::cout << "STEQR test ... ";

#ifndef TEUCHOSNUMERICS_DISABLE_STEQR_TEST

const int n_steqr = 10;
std::vector<MagnitudeType> diagonal(n_steqr);
std::vector<MagnitudeType> subdiagonal(n_steqr-1);
Expand Down Expand Up @@ -254,6 +256,12 @@ int main(int argc, char* argv[])
numberFailedTests++;
}

#else // TEUCHOSNUMERICS_DISABLE_STEQR_TEST

if (verbose) std::cout << "SKIPPED!\n";

#endif // TEUCHOSNUMERICS_DISABLE_STEQR_TEST

if(numberFailedTests > 0)
{
if (verbose) {
Expand Down

0 comments on commit f8e692d

Please sign in to comment.