[all-commits] [llvm/llvm-project] d2a6ec: [clangd] Use CommandMangler in TestTU

[kadir çetinkaya via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d2a6ec8eae75b60081e081eeb37022839f4f2d04
      https://github.com/llvm/llvm-project/commit/d2a6ec8eae75b60081e081eeb37022839f4f2d04
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2021-07-23 (Fri, 23 Jul 2021)

  Changed paths:
    M clang-tools-extra/clangd/unittests/TestTU.cpp
    M clang/lib/Frontend/CreateInvocationFromCommandLine.cpp

  Log Message:
  -----------
  [clangd] Use CommandMangler in TestTU

This makes testing setup look closer to production.

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


  Commit: 1c0d0085bcaaf27cc8d9492eb3c5c05058e54b8e
      https://github.com/llvm/llvm-project/commit/1c0d0085bcaaf27cc8d9492eb3c5c05058e54b8e
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2021-07-23 (Fri, 23 Jul 2021)

  Changed paths:
    M clang-tools-extra/clangd/CompileCommands.cpp
    M clang-tools-extra/clangd/CompileCommands.h
    M clang-tools-extra/clangd/Compiler.cpp
    M clang-tools-extra/clangd/unittests/CompileCommandsTests.cpp
    M clang-tools-extra/clangd/unittests/CompilerTests.cpp

  Log Message:
  -----------
  [clangd] Get rid of arg adjusters in CommandMangler

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


  Commit: 7cc8a8e3849dc4044cc799e2c1f6cc241b851b70
      https://github.com/llvm/llvm-project/commit/7cc8a8e3849dc4044cc799e2c1f6cc241b851b70
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2021-07-23 (Fri, 23 Jul 2021)

  Changed paths:
    M clang-tools-extra/clangd/CompileCommands.cpp
    M clang-tools-extra/clangd/test/did-change-configuration-params.test
    M clang-tools-extra/clangd/unittests/BackgroundIndexTests.cpp
    M clang-tools-extra/clangd/unittests/CompileCommandsTests.cpp

  Log Message:
  -----------
  [clangd] Canonicalize compile flags before applying edits

Pushes input for the compile action to the end while separating with a
`--` before applying other manglings. This ensures edits that effect only the
arguments that come after them works, like changing parse language via -x.

Fixes https://github.com/clangd/clangd/issues/555.

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


  Commit: ba5dd945ad9124f24452987be64040a6ea6cd25e
      https://github.com/llvm/llvm-project/commit/ba5dd945ad9124f24452987be64040a6ea6cd25e
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2021-07-23 (Fri, 23 Jul 2021)

  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-tools-extra/clangd/unittests/TestTU.cpp

  Log Message:
  -----------
  [clangd] Adjust compile flags to contain only the requested file as input

Depends on D106527.

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


Compare: https://github.com/llvm/llvm-project/compare/f86570cd5241...ba5dd945ad91