[all-commits] [llvm/llvm-project] 34de7d: [clangd] Move standard options adaptor to CommandM...

Dmitry Polukhin via All-commits all-commits at lists.llvm.org
Mon Mar 13 06:11:00 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 34de7da6246cdfa6ff6f3d3c514583cddc0a10ec
      https://github.com/llvm/llvm-project/commit/34de7da6246cdfa6ff6f3d3c514583cddc0a10ec
  Author: Dmitry Polukhin <dmitry.polukhin at gmail.com>
  Date:   2023-03-13 (Mon, 13 Mar 2023)

  Changed paths:
    M clang-tools-extra/clangd/CompileCommands.cpp
    M clang-tools-extra/clangd/CompileCommands.h
    M clang-tools-extra/clangd/GlobalCompilationDatabase.cpp
    M clang-tools-extra/clangd/test/did-change-configuration-params.test
    M clang-tools-extra/clangd/unittests/CompileCommandsTests.cpp
    M clang/include/clang/Tooling/Tooling.h
    M clang/lib/Tooling/ExpandResponseFilesCompilationDatabase.cpp
    M clang/lib/Tooling/Tooling.cpp

  Log Message:
  -----------
  [clangd] Move standard options adaptor to CommandMangler

There is a discrepancy between how clangd processes CDB loaded from
JSON file on disk and pushed via LSP. Thus the same CDB pushed via
LSP protocol may not work as expected. Some difference between these two
paths is expected but we still need to insert driver mode and target from
binary name and expand response files.

Test Plan: check-clang-tools

Differential Revision: https://reviews.llvm.org/D143436




More information about the All-commits mailing list