diff options
author | Pavel Labath <pavel@labath.sk> | 2025-04-22 10:14:03 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-04-22 10:14:03 +0200 |
commit | a22ad659cd0665669d89fae174f9e6a83d1a446d (patch) | |
tree | c08025b7137e3b7d89e5c24ae40869a6945043f9 /lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp | |
parent | 8bc0d4d1ccb708d20413876258885a48d3ca6b3d (diff) | |
download | llvm-a22ad659cd0665669d89fae174f9e6a83d1a446d.zip llvm-a22ad659cd0665669d89fae174f9e6a83d1a446d.tar.gz llvm-a22ad659cd0665669d89fae174f9e6a83d1a446d.tar.bz2 |
[lldb/cmake] Normalize use of HAVE_LIBCOMPRESSION (#135528)
I *think* this was the reason behind the failures in
2fd860c1f559c0b0be66cc000e38270a04d0a1a3: the clang include tool showed
the Config.h headers as unused, and because the macro was referenced
through an `#ifdef`, its removal didn't cause build failures. Switching
to `#cmakedefine01` + `#if` should make sure this does not happen again.
According to D48977, the `#ifndef`+`#cmakedefine` patterns is due to
some files redefining the macro themselves. I no longer see any such
files in the source tree (there also were no files like that in the
source tree at the revision mentioned, but the macro *was* defined in
the hand-maintained XCode project we had at the time).
Diffstat (limited to 'lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp')
-rw-r--r-- | lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp index 501670d..748f95c 100644 --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp @@ -41,7 +41,7 @@ #include "llvm/Config/llvm-config.h" // for LLVM_ENABLE_ZLIB #include "llvm/Support/JSON.h" -#if defined(HAVE_LIBCOMPRESSION) +#if HAVE_LIBCOMPRESSION #include <compression.h> #endif @@ -1104,7 +1104,7 @@ void GDBRemoteCommunicationClient::MaybeEnableCompression( CompressionType avail_type = CompressionType::None; llvm::StringRef avail_name; -#if defined(HAVE_LIBCOMPRESSION) +#if HAVE_LIBCOMPRESSION if (avail_type == CompressionType::None) { for (auto compression : supported_compressions) { if (compression == "lzfse") { @@ -1114,9 +1114,6 @@ void GDBRemoteCommunicationClient::MaybeEnableCompression( } } } -#endif - -#if defined(HAVE_LIBCOMPRESSION) if (avail_type == CompressionType::None) { for (auto compression : supported_compressions) { if (compression == "zlib-deflate") { @@ -1140,7 +1137,7 @@ void GDBRemoteCommunicationClient::MaybeEnableCompression( } #endif -#if defined(HAVE_LIBCOMPRESSION) +#if HAVE_LIBCOMPRESSION if (avail_type == CompressionType::None) { for (auto compression : supported_compressions) { if (compression == "lz4") { @@ -1150,9 +1147,6 @@ void GDBRemoteCommunicationClient::MaybeEnableCompression( } } } -#endif - -#if defined(HAVE_LIBCOMPRESSION) if (avail_type == CompressionType::None) { for (auto compression : supported_compressions) { if (compression == "lzma") { |