]> https://gitweb.dealii.org/ - dealii.git/commitdiff
CMake: remove support for ctest_memcheck() and run_memorycheck.cmake 15716/head
authorMatthias Maier <tamiko@43-1.org>
Tue, 11 Jul 2023 05:59:32 +0000 (00:59 -0500)
committerMatthias Maier <tamiko@43-1.org>
Tue, 11 Jul 2023 06:04:04 +0000 (01:04 -0500)
Instead of using valgrind it is much better to use compiler sanitizers
such as "-fsanitize=address,undefined,leak" with clang.

doc/developers/testsuite.html
tests/run_memorycheck.cmake [deleted file]
tests/run_testsuite.cmake

index 0230c029c027ad56f345c717ba30d7cd99efd020..cce7ca3032a5f9c466dd7dc5bdae8eafe94784f9 100644 (file)
@@ -306,33 +306,6 @@ xx: ===============================    OUTPUT END   ============================
     </p>
 
 
-    <a name="memcheck"></a>
-     <h3>Running with valgrind</h3>
-
-     <p>
-       The testsuite can also be run using <a href="http://valgrind.org/">valgrind</a>
-       to check for <i>memory corruption</i> in the library.
-     </p>
-
-     <p>
-       You can do so by invoking
- <pre>
- ctest -DMEMORYCHECK=ON <...> -S ../tests/run_testsuite.cmake
- </pre>
-       when running the testsuite, or directly by
- <pre>
- ctest <...> -S ../tests/run_memorycheck.cmake
- </pre>
-     </p>
-
-     <p>
-       At the end of all of this, results will be shown in a separate section
-       "Dynamic Analysis" at the
-       <a href="https://cdash.dealii.org/index.php?project=deal.II&display=project"
-       target="_top">deal.II cdash site</a>.
-     </p>
-
-
     <a name="coverage"></a>
     <h3>Generating coverage information</h3>
 
diff --git a/tests/run_memorycheck.cmake b/tests/run_memorycheck.cmake
deleted file mode 100644 (file)
index 8ad9716..0000000
+++ /dev/null
@@ -1,3 +0,0 @@
-set(TRACK "Experimental")
-set(MEMORYCHECK TRUE)
-include(${CMAKE_CURRENT_LIST_DIR}/run_testsuite.cmake)
index 9d8c721d44c002bc561fe4662d269afbc05923fc..e30878300e26c2f4ec975b85d1ea475a2a2c48cb 100644 (file)
 #     ctest_coverage() stage will be run. Test results must go into the
 #     "Experimental" section.
 #
-#   MEMORYCHECK
-#     - If set to ON the CTEST_MEMORYCHECK() stage will be run.
-#     Test results must go into the "Experimental" section.
-#
 #   MAKEOPTS
 #     - Additional options that will be passed directly to make (or ninja).
 #
@@ -436,35 +432,6 @@ endif()
 message("-- COVERAGE:               ${COVERAGE}")
 
 
-#
-# Setup memcheck:
-#
-
-if(MEMORYCHECK)
-  if(NOT TRACK MATCHES "Experimental")
-    message(FATAL_ERROR "
-TRACK must be set to  \"Experimental\" if Memcheck is enabled via
-MEMORYCHECK=TRUE.
-"
-      )
-  endif()
-
-  find_program(MEMORYCHECK_COMMAND NAMES valgrind)
-  if(MEMORYCOMMAND MATCHES "-NOTFOUND")
-    message(FATAL_ERROR "
-Memcheck enabled but could not find the valgrind executable. Please install
-valgrind.
-"
-       )
-  endif()
-
-  set(CTEST_MEMORYCHECK_COMMAND "${MEMORYCHECK_COMMAND}")
-endif()
-
-message("-- MEMORYCHECK:               ${MEMORYCHECK}")
-
-
-
 macro(create_targetdirectories_txt)
   #
   # It gets tricky: Fake a TargetDirectories.txt containing _all_ target
@@ -563,13 +530,9 @@ if("${_res}" STREQUAL "0")
     if(DEAL_II_MSVC)
       set(CTEST_BUILD_CONFIGURATION "${JOB_BUILD_CONFIGURATION}")
     endif()
-    if(MEMORYCHECK)
-      message("-- Running ctest_memcheck()")
-      ctest_memcheck()
-    else()
-      message("-- Running CTEST_TESTS()")
-      ctest_test()
-    endif()
+
+    message("-- Running CTEST_TESTS()")
+    ctest_test()
 
     if(COVERAGE)
       CREATE_TARGETDIRECTORIES_TXT()

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.