[PATCH] D71029: [clangd] (take 2) Try harder to find a plausible `clang` as argv0, particularly on Mac.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 4 11:08:58 PST 2019


sammccall created this revision.
sammccall added reviewers: kbobyrev, ilya-biryukov.
sammccall added a project: clang.
Herald added subscribers: ormris, mgorny.

This was originally committed in 88bccded8fa169481fa367debf5ec615640635a1 <https://reviews.llvm.org/rG88bccded8fa169481fa367debf5ec615640635a1>,
and reverted in 93f77617abba512d2861e2fc50ce385883f587b6 <https://reviews.llvm.org/rG93f77617abba512d2861e2fc50ce385883f587b6>.

This version is now much more testable: the "detect toolchain properties" part
is still not tested but also not active in tests.
All the command manipulation based on the detected properties is
directly tested, and also not active in other tests.

Fixes https://github.com/clangd/clangd/issues/211
Fixes https://github.com/clangd/clangd/issues/178


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D71029

Files:
  clang-tools-extra/clangd/CMakeLists.txt
  clang-tools-extra/clangd/ClangdLSPServer.cpp
  clang-tools-extra/clangd/CompileCommands.cpp
  clang-tools-extra/clangd/CompileCommands.h
  clang-tools-extra/clangd/GlobalCompilationDatabase.cpp
  clang-tools-extra/clangd/GlobalCompilationDatabase.h
  clang-tools-extra/clangd/unittests/BackgroundIndexTests.cpp
  clang-tools-extra/clangd/unittests/CMakeLists.txt
  clang-tools-extra/clangd/unittests/ClangdTests.cpp
  clang-tools-extra/clangd/unittests/CompileCommandsTests.cpp
  clang-tools-extra/clangd/unittests/GlobalCompilationDatabaseTests.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71029.232181.patch
Type: text/x-patch
Size: 27644 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191204/8557212f/attachment-0001.bin>


More information about the cfe-commits mailing list