[PATCH] D73726: Rename llvm-prettyprinters to visualizer-tests.

[Christian Sigg via Phabricator via llvm-commits]

csigg updated this revision to Diff 242028.
csigg added a comment.

Revert back to llvm-prettyprinters directory.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73726/new/

https://reviews.llvm.org/D73726

Files:
  debuginfo-tests/CMakeLists.txt
  debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.cpp
  debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.gdb
  debuginfo-tests/llvm-prettyprinters/gdb/prettyprinters.cpp
  debuginfo-tests/llvm-prettyprinters/gdb/prettyprinters.gdb


Index: debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.gdb
===================================================================
--- debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.gdb
+++ debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.gdb
@@ -1,4 +1,4 @@
-# RUN: gdb -q -batch -n -iex 'source %llvm_src_root/utils/gdb-scripts/prettyprinters.py' -x %s %llvm_tools_dir/prettyprinters | FileCheck %s
+# RUN: gdb -q -batch -n -iex 'source %llvm_src_root/utils/gdb-scripts/prettyprinters.py' -x %s %llvm_tools_dir/check-gdb-llvm-support | FileCheck %s
 
 break main
 run
Index: debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.cpp
===================================================================
--- debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.cpp
+++ debuginfo-tests/llvm-prettyprinters/gdb/llvm-support.cpp
@@ -47,4 +47,8 @@
   return Result;
 }();
 
+// Check expected instances to avoid compile errors.
+auto CheckExpectedValue = static_cast<bool>(ExpectedValue);
+auto CheckExpectedError = static_cast<bool>(ExpectedError);
+
 int main() { return 0; }
Index: debuginfo-tests/CMakeLists.txt
===================================================================
--- debuginfo-tests/CMakeLists.txt
+++ debuginfo-tests/CMakeLists.txt
@@ -2,10 +2,10 @@
 # various types of debug info, and then run those programs under a debugger
 # such as GDB or LLDB to verify the results.
 
-add_llvm_executable(prettyprinters
-        llvm-prettyprinters/gdb/prettyprinters.cpp
+add_llvm_executable(check-gdb-llvm-support
+        llvm-prettyprinters/gdb/llvm-support.cpp
 )
-target_link_libraries(prettyprinters PRIVATE LLVMSupport)
+target_link_libraries(check-gdb-llvm-support PRIVATE LLVMSupport)
 
 set(DEBUGINFO_TESTS_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR})
 set(DEBUGINFO_TESTS_BINARY_DIR ${CMAKE_CURRENT_BINARY_DIR})
@@ -15,8 +15,8 @@
   FileCheck
   count
   llvm-objdump
+  check-gdb-llvm-support
   not
-  prettyprinters
   )
 
 # The Windows builder scripts pass -fuse-ld=lld.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D73726.242028.patch
Type: text/x-patch
Size: 2001 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200203/b23de944/attachment.bin>