[all-commits] [llvm/llvm-project] b8d688: [NFC] Add explicit #include llvm-config.h where it...

[Daniil Fukalov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b8d6885ff67efbc3142a2b49506ed0cc2b95e054
      https://github.com/llvm/llvm-project/commit/b8d6885ff67efbc3142a2b49506ed0cc2b95e054
  Author: Daniil Fukalov <dfukalov at gmail.com>
  Date:   2024-09-06 (Fri, 06 Sep 2024)

  Changed paths:
    M clang-tools-extra/clangd/Feature.cpp
    M clang-tools-extra/clangd/unittests/ClangdTests.cpp
    M clang-tools-extra/clangd/unittests/CompileCommandsTests.cpp
    M clang-tools-extra/clangd/unittests/SerializationTests.cpp
    M clang/include/clang/Interpreter/Value.h
    M clang/lib/Driver/ToolChains/Cuda.cpp
    M clang/lib/Driver/ToolChains/MinGW.cpp
    M clang/lib/Driver/ToolChains/WebAssembly.cpp
    M clang/lib/Frontend/FrontendActions.cpp
    M clang/tools/driver/driver.cpp
    M clang/unittests/Driver/GCCVersionTest.cpp

  Log Message:
  -----------
  [NFC] Add explicit #include llvm-config.h where its macros are used, clang part. (#107301)

(this is clang related part)

Without these explicit includes, removing other headers, who implicitly
include llvm-config.h, may have non-trivial side effects. For example,
`clagd` may report even `llvm-config.h` as "no used" in case it defines
a macro, that is explicitly used with #ifdef. It is actually amplified
with different build configs which use different set of macros.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications