[all-commits] [llvm/llvm-project] 345cc4: [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: 345cc47ba7a28811ae4ec7d113059ccb39c500a3
      https://github.com/llvm/llvm-project/commit/345cc47ba7a28811ae4ec7d113059ccb39c500a3
  Author: Daniil Fukalov <dfukalov at gmail.com>
  Date:   2024-09-09 (Mon, 09 Sep 2024)

  Changed paths:
    M lldb/source/API/SBDebugger.cpp
    M lldb/source/Host/common/Host.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
    M lldb/source/Plugins/SymbolFile/CTF/SymbolFileCTF.cpp
    M lldb/source/Plugins/SymbolFile/PDB/SymbolFilePDB.cpp
    M lldb/unittests/Host/MainLoopTest.cpp

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

(this is lldb part)

Without these explicit includes, removing other headers, who implicitly
include llvm-config.h, may have non-trivial side effects. For example,
`clangd` 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