intel/llvm
1
0
Fork 0
mirror of https://github.com/intel/llvm.git synced 2026-01-25 01:07:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

[lldb/cmake] Normalize use of HAVE_LIBCOMPRESSION (#135528)

Browse Source 
I *think* this was the reason behind the failures in
2fd860c1f5: 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).
This commit is contained in:
Pavel Labath
2025-04-22 10:14:03 +02:00
committed by GitHub
parent 8bc0d4d1cc
commit a22ad659cd
4 changed files with 8 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lldb/include/lldb/Host/Config.h.cmake
View File

@@ -23,9 +23,7 @@
#cmakedefine01 HAVE_NR_PROCESS_VM_READV
#ifndef HAVE_LIBCOMPRESSION
#cmakedefine HAVE_LIBCOMPRESSION
#endif
#cmakedefine01 HAVE_LIBCOMPRESSION
#cmakedefine01 LLDB_ENABLE_POSIX

6
lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp
View File

@@ -41,7 +41,7 @@
#define DEBUGSERVER_BASENAME "lldb-server"
#endif
#if defined(HAVE_LIBCOMPRESSION)
#if HAVE_LIBCOMPRESSION
#include <compression.h>
#endif
@@ -72,7 +72,7 @@ GDBRemoteCommunication::~GDBRemoteCommunication() {
Disconnect();
}
#if defined(HAVE_LIBCOMPRESSION)
#if HAVE_LIBCOMPRESSION
if (m_decompression_scratch)
free (m_decompression_scratch);
#endif
@@ -509,7 +509,7 @@ bool GDBRemoteCommunication::DecompressPacket() {
}
}
#if defined(HAVE_LIBCOMPRESSION)
#if HAVE_LIBCOMPRESSION
if (m_compression_type == CompressionType::ZlibDeflate ||
m_compression_type == CompressionType::LZFSE ||
m_compression_type == CompressionType::LZ4 ||

2
lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.h
View File

@@ -214,7 +214,7 @@ private:
HostThread m_listen_thread;
std::string m_listen_url;
#if defined(HAVE_LIBCOMPRESSION)
#if HAVE_LIBCOMPRESSION
CompressionType m_decompression_scratch_type = CompressionType::None;
void *m_decompression_scratch = nullptr;
#endif

12
lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
View File

@@ -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") {