[PATCH] D51987: [clangd] Rename global-symbol-builder to clangd-symbol-builder.

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov updated this revision to Diff 165085.
ilya-biryukov added a comment.

- Rebase, updated the added test


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D51987

Files:
  clangd/CMakeLists.txt
  clangd/global-symbol-builder/CMakeLists.txt
  clangd/global-symbol-builder/GlobalSymbolBuilderMain.cpp
  clangd/symbol-builder/CMakeLists.txt
  clangd/symbol-builder/SymbolBuilderMain.cpp
  test/CMakeLists.txt
  test/clangd/index-tools.test


Index: test/clangd/index-tools.test
===================================================================
--- test/clangd/index-tools.test
+++ test/clangd/index-tools.test
@@ -1,3 +1,3 @@
-# RUN: global-symbol-builder %p/Inputs/BenchmarkSource.cpp -- -I%p/Inputs > %t.index
+# RUN: clangd-symbol-builder %p/Inputs/BenchmarkSource.cpp -- -I%p/Inputs > %t.index
 # FIXME: By default, benchmarks are excluded from the list of default targets hence not built. Find a way to depend on benchmarks to run the next command.
 # RUN: if [ -f %clangd-benchmark-dir/IndexBenchmark ]; then %clangd-benchmark-dir/IndexBenchmark %t.index %p/Inputs/requests.log --benchmark_min_time=0.01 ; fi
Index: test/CMakeLists.txt
===================================================================
--- test/CMakeLists.txt
+++ test/CMakeLists.txt
@@ -55,7 +55,7 @@
 
   # These individual tools have no tests, add them here to make them compile
   # together with check-clang-tools, so that we won't break them in the future.
-  global-symbol-builder
+  clangd-symbol-builder
 
   # Unit tests
   ExtraToolsUnitTests
Index: clangd/symbol-builder/SymbolBuilderMain.cpp
===================================================================
--- clangd/symbol-builder/SymbolBuilderMain.cpp
+++ clangd/symbol-builder/SymbolBuilderMain.cpp
@@ -225,11 +225,11 @@
   Example usage for building index for the whole project using CMake compile
   commands:
 
-  $ global-symbol-builder --executor=all-TUs compile_commands.json > index.yaml
+  $ clangd-symbol-builder --executor=all-TUs compile_commands.json > index.yaml
 
   Example usage for file sequence index without flags:
 
-  $ global-symbol-builder File1.cpp File2.cpp ... FileN.cpp > index.yaml
+  $ clangd-symbol-builder File1.cpp File2.cpp ... FileN.cpp > index.yaml
 
   Note: only symbols from header files will be collected.
   )";
Index: clangd/symbol-builder/CMakeLists.txt
===================================================================
--- clangd/symbol-builder/CMakeLists.txt
+++ clangd/symbol-builder/CMakeLists.txt
@@ -4,11 +4,11 @@
     Support
     )
 
-add_clang_executable(global-symbol-builder
-  GlobalSymbolBuilderMain.cpp
+add_clang_executable(clangd-symbol-builder
+  SymbolBuilderMain.cpp
   )
 
-target_link_libraries(global-symbol-builder
+target_link_libraries(clangd-symbol-builder
   PRIVATE
   clangAST
   clangIndex
Index: clangd/CMakeLists.txt
===================================================================
--- clangd/CMakeLists.txt
+++ clangd/CMakeLists.txt
@@ -73,7 +73,7 @@
   add_subdirectory(fuzzer)
 endif()
 add_subdirectory(tool)
-add_subdirectory(global-symbol-builder)
+add_subdirectory(symbol-builder)
 add_subdirectory(index/dex/dexp)
 
 if (LLVM_INCLUDE_BENCHMARKS)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51987.165085.patch
Type: text/x-patch
Size: 2737 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180912/0dbe66c9/attachment.bin>