[PATCH] D68477: Add an off-by-default option to enable testing for gdb pretty printers.
Sterling Augustine via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 4 15:15:45 PDT 2019
This revision was automatically updated to reflect the committed changes.
Closed by commit rL373796: Add an off-by-default option to enable testing for gdb pretty printers. (authored by saugustine, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D68477?vs=223260&id=223320#toc
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68477/new/
https://reviews.llvm.org/D68477
Files:
libcxx/trunk/CMakeLists.txt
libcxx/trunk/test/CMakeLists.txt
Index: libcxx/trunk/test/CMakeLists.txt
===================================================================
--- libcxx/trunk/test/CMakeLists.txt
+++ libcxx/trunk/test/CMakeLists.txt
@@ -60,12 +60,16 @@
message(FATAL_ERROR "Expected LIBCXX_TEST_DEPS to be defined")
endif()
-find_program(LIBCXX_GDB gdb)
-if (LIBCXX_GDB)
- set(LIBCXX_GDB "${LIBCXX_GDB}")
- message(STATUS "gdb found: ${LIBCXX_GDB}")
-else()
- message(STATUS "gdb not found. Disabling dependent tests.")
+# Turn this on by default when the pretty printers are python3
+# compatible.
+if(LIBCXX_TEST_GDB_PRETTY_PRINTERS)
+ find_program(LIBCXX_GDB gdb)
+ if (LIBCXX_GDB)
+ set(LIBCXX_GDB "${LIBCXX_GDB}")
+ message(STATUS "gdb found: ${LIBCXX_GDB}")
+ else()
+ message(STATUS "gdb not found. Disabling dependent tests.")
+ endif()
endif()
if (LIBCXX_INCLUDE_TESTS)
Index: libcxx/trunk/CMakeLists.txt
===================================================================
--- libcxx/trunk/CMakeLists.txt
+++ libcxx/trunk/CMakeLists.txt
@@ -81,6 +81,7 @@
${ENABLE_FILESYSTEM_DEFAULT})
option(LIBCXX_INCLUDE_TESTS "Build the libc++ tests." ${LLVM_INCLUDE_TESTS})
option(LIBCXX_ENABLE_PARALLEL_ALGORITHMS "Enable the parallel algorithms library. This requires the PSTL to be available." OFF)
+option(LIBCXX_TEST_GDB_PRETTY_PRINTERS "Test gdb pretty printers." OFF)
# Benchmark options -----------------------------------------------------------
option(LIBCXX_INCLUDE_BENCHMARKS "Build the libc++ benchmarks and their dependencies" ON)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68477.223320.patch
Type: text/x-patch
Size: 1528 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191004/bc5d1490/attachment.bin>
More information about the llvm-commits
mailing list