Skip to content

Commit

Permalink
[lldb] Qualify includes of Properties[Enum].inc files. NFC
Browse files Browse the repository at this point in the history
Summary:
This is a bit more explicit, and makes it possible to build LLDB without
varying the -I lines per-directory.
(The latter is useful because many build systems only allow this to be
configured per-library, and LLDB is insufficiently layered to be split into
multiple libraries on stricter build systems).

(My comment on D65185 has some more context)

Reviewers: JDevlieghere, labath, chandlerc, jdoerfert

Reviewed By: labath

Subscribers: mgorny, lldb-commits

Tags: #lldb

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

Patch by Sam McCall!

llvm-svn: 367241
  • Loading branch information
rupprecht committed Jul 29, 2019
1 parent 962c03f commit 6a253d3
Show file tree
Hide file tree
Showing 34 changed files with 71 additions and 71 deletions.
8 changes: 4 additions & 4 deletions lldb/source/Core/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(CoreProperties.inc -gen-lldb-property-defs
SOURCE CoreProperties.td
TARGET LLDBCorePropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(CorePropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE CoreProperties.td
TARGET LLDBCorePropertiesEnumGen)

set(LLDB_CURSES_LIBS)
Expand Down
File renamed without changes.
4 changes: 2 additions & 2 deletions lldb/source/Core/Debugger.cpp
Expand Up @@ -203,11 +203,11 @@ static constexpr OptionEnumValueElement s_stop_show_column_values[] = {
{eStopShowColumnNone, "none", "Do not highlight the stop column."}};

#define LLDB_PROPERTIES_debugger
#include "Properties.inc"
#include "CoreProperties.inc"

enum {
#define LLDB_PROPERTIES_debugger
#include "PropertiesEnum.inc"
#include "CorePropertiesEnum.inc"
};

LoadPluginCallbackType Debugger::g_load_plugin_callback = nullptr;
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Core/ModuleList.cpp
Expand Up @@ -66,11 +66,11 @@ using namespace lldb_private;
namespace {

#define LLDB_PROPERTIES_modulelist
#include "Properties.inc"
#include "CoreProperties.inc"

enum {
#define LLDB_PROPERTIES_modulelist
#include "PropertiesEnum.inc"
#include "CorePropertiesEnum.inc"
};

} // namespace
Expand Down
10 changes: 5 additions & 5 deletions lldb/source/Interpreter/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(InterpreterProperties.inc -gen-lldb-property-defs
SOURCE InterpreterProperties.td
TARGET LLDBInterpreterPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(InterpreterPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE InterpreterProperties.td
TARGET LLDBInterpreterPropertiesEnumGen)

add_lldb_library(lldbInterpreter
Expand Down Expand Up @@ -70,4 +70,4 @@ add_dependencies(lldbInterpreter

if (NOT LLDB_DISABLE_LIBEDIT)
target_include_directories(lldbInterpreter PRIVATE ${libedit_INCLUDE_DIRS})
endif()
endif()
4 changes: 2 additions & 2 deletions lldb/source/Interpreter/CommandInterpreter.cpp
Expand Up @@ -90,11 +90,11 @@ static constexpr const char *InitFileWarning =
"accept the security risk.";

#define LLDB_PROPERTIES_interpreter
#include "Properties.inc"
#include "InterpreterProperties.inc"

enum {
#define LLDB_PROPERTIES_interpreter
#include "PropertiesEnum.inc"
#include "InterpreterPropertiesEnum.inc"
};

ConstString &CommandInterpreter::GetStaticBroadcasterClass() {
Expand Down
File renamed without changes.
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(DynamicLoaderDarwinKernelProperties.inc -gen-lldb-property-defs
SOURCE DynamicLoaderDarwinKernelProperties.td
TARGET LLDBPluginDynamicLoaderDarwinKernelPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(DynamicLoaderDarwinKernelPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE DynamicLoaderDarwinKernelProperties.td
TARGET LLDBPluginDynamicLoaderDarwinKernelPropertiesEnumGen)

add_lldb_library(lldbPluginDynamicLoaderDarwinKernel PLUGIN
Expand Down
Expand Up @@ -73,11 +73,11 @@ static constexpr OptionEnumValueElement g_kaslr_kernel_scan_enum_values[] = {
"on 32-bit targets)."}};

#define LLDB_PROPERTIES_dynamicloaderdarwinkernel
#include "Properties.inc"
#include "DynamicLoaderDarwinKernelProperties.inc"

enum {
#define LLDB_PROPERTIES_dynamicloaderdarwinkernel
#include "PropertiesEnum.inc"
#include "DynamicLoaderDarwinKernelPropertiesEnum.inc"
};

class DynamicLoaderDarwinKernelProperties : public Properties {
Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Plugins/JITLoader/GDB/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(JITLoaderGDBProperties.inc -gen-lldb-property-defs
SOURCE JITLoaderGDBProperties.td
TARGET LLDBPluginJITLoaderGDBPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(JITLoaderGDBPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE JITLoaderGDBProperties.td
TARGET LLDBPluginJITLoaderGDBPropertiesEnumGen)

add_lldb_library(lldbPluginJITLoaderGDB PLUGIN
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Plugins/JITLoader/GDB/JITLoaderGDB.cpp
Expand Up @@ -67,11 +67,11 @@ static constexpr OptionEnumValueElement g_enable_jit_loader_gdb_enumerators[] =
};

#define LLDB_PROPERTIES_jitloadergdb
#include "Properties.inc"
#include "JITLoaderGDBProperties.inc"

enum {
#define LLDB_PROPERTIES_jitloadergdb
#include "PropertiesEnum.inc"
#include "JITLoaderGDBPropertiesEnum.inc"
ePropertyEnableJITBreakpoint
};

Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Plugins/Platform/MacOSX/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(PlatformMacOSXProperties.inc -gen-lldb-property-defs
SOURCE PlatformMacOSXProperties.td
TARGET LLDBPluginPlatformMacOSXPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(PlatformMacOSXPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE PlatformMacOSXProperties.td
TARGET LLDBPluginPlatformMacOSXPropertiesEnumGen)

list(APPEND PLUGIN_PLATFORM_MACOSX_SOURCES
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Plugins/Platform/MacOSX/PlatformDarwinKernel.cpp
Expand Up @@ -178,11 +178,11 @@ const char *PlatformDarwinKernel::GetDescriptionStatic() {
/// Code to handle the PlatformDarwinKernel settings

#define LLDB_PROPERTIES_platformdarwinkernel
#include "Properties.inc"
#include "PlatformMacOSXProperties.inc"

enum {
#define LLDB_PROPERTIES_platformdarwinkernel
#include "PropertiesEnum.inc"
#include "PlatformMacOSXPropertiesEnum.inc"
};

class PlatformDarwinKernelProperties : public Properties {
Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Plugins/Process/MacOSX-Kernel/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(ProcessKDPProperties.inc -gen-lldb-property-defs
SOURCE ProcessKDPProperties.td
TARGET LLDBPluginProcessMacOSXKernelPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(ProcessKDPPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE ProcessKDPProperties.td
TARGET LLDBPluginProcessMacOSXKernelPropertiesEnumGen)

add_lldb_library(lldbPluginProcessMacOSXKernel PLUGIN
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp
Expand Up @@ -53,11 +53,11 @@ using namespace lldb_private;
namespace {

#define LLDB_PROPERTIES_processkdp
#include "Properties.inc"
#include "ProcessKDPProperties.inc"

enum {
#define LLDB_PROPERTIES_processkdp
#include "PropertiesEnum.inc"
#include "ProcessKDPPropertiesEnum.inc"
};

class PluginProperties : public Properties {
Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Plugins/Process/gdb-remote/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(ProcessGDBRemoteProperties.inc -gen-lldb-property-defs
SOURCE ProcessGDBRemoteProperties.td
TARGET LLDBPluginProcessGDBRemotePropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(ProcessGDBRemotePropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE ProcessGDBRemoteProperties.td
TARGET LLDBPluginProcessGDBRemotePropertiesEnumGen)

if (CMAKE_SYSTEM_NAME MATCHES "Darwin")
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
Expand Up @@ -111,11 +111,11 @@ void DumpProcessGDBRemotePacketHistory(void *p, const char *path) {
namespace {

#define LLDB_PROPERTIES_processgdbremote
#include "Properties.inc"
#include "ProcessGDBRemoteProperties.inc"

enum {
#define LLDB_PROPERTIES_processgdbremote
#include "PropertiesEnum.inc"
#include "ProcessGDBRemotePropertiesEnum.inc"
};

class PluginProperties : public Properties {
Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Plugins/StructuredData/DarwinLog/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(StructuredDataDarwinLogProperties.inc -gen-lldb-property-defs
SOURCE StructuredDataDarwinLogProperties.td
TARGET LLDBPluginStructuredDataDarwinLogPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(StructuredDataDarwinLogPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE StructuredDataDarwinLogProperties.td
TARGET LLDBPluginStructuredDataDarwinLogPropertiesEnumGen)

add_lldb_library(lldbPluginStructuredDataDarwinLog PLUGIN
Expand Down
Expand Up @@ -105,11 +105,11 @@ void SetGlobalEnableOptions(const DebuggerSP &debugger_sp,
/// Code to handle the StructuredDataDarwinLog settings

#define LLDB_PROPERTIES_darwinlog
#include "Properties.inc"
#include "StructuredDataDarwinLogProperties.inc"

enum {
#define LLDB_PROPERTIES_darwinlog
#include "PropertiesEnum.inc"
#include "StructuredDataDarwinLogPropertiesEnum.inc"
};

class StructuredDataDarwinLogProperties : public Properties {
Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Plugins/SymbolFile/DWARF/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(SymbolFileDWARFProperties.inc -gen-lldb-property-defs
SOURCE SymbolFileDWARFProperties.td
TARGET LLDBPluginSymbolFileDWARFPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(SymbolFileDWARFPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE SymbolFileDWARFProperties.td
TARGET LLDBPluginSymbolFileDWARFPropertiesEnumGen)

add_lldb_library(lldbPluginSymbolFileDWARF PLUGIN
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp
Expand Up @@ -114,11 +114,11 @@ using namespace lldb_private;
namespace {

#define LLDB_PROPERTIES_symbolfiledwarf
#include "Properties.inc"
#include "SymbolFileDWARFProperties.inc"

enum {
#define LLDB_PROPERTIES_symbolfiledwarf
#include "PropertiesEnum.inc"
#include "SymbolFileDWARFPropertiesEnum.inc"
};

class PluginProperties : public Properties {
Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Target/CMakeLists.txt
@@ -1,9 +1,9 @@
lldb_tablegen(Properties.inc -gen-lldb-property-defs
SOURCE Properties.td
lldb_tablegen(TargetProperties.inc -gen-lldb-property-defs
SOURCE TargetProperties.td
TARGET LLDBTargetPropertiesGen)

lldb_tablegen(PropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE Properties.td
lldb_tablegen(TargetPropertiesEnum.inc -gen-lldb-property-enum-defs
SOURCE TargetProperties.td
TARGET LLDBTargetPropertiesEnumGen)

add_lldb_library(lldbTarget
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Target/Platform.cpp
Expand Up @@ -64,11 +64,11 @@ const char *Platform::GetHostPlatformName() { return "host"; }
namespace {

#define LLDB_PROPERTIES_platform
#include "Properties.inc"
#include "TargetProperties.inc"

enum {
#define LLDB_PROPERTIES_platform
#include "PropertiesEnum.inc"
#include "TargetPropertiesEnum.inc"
};

} // namespace
Expand Down
4 changes: 2 additions & 2 deletions lldb/source/Target/Process.cpp
Expand Up @@ -113,11 +113,11 @@ class ProcessOptionValueProperties : public OptionValueProperties {
};

#define LLDB_PROPERTIES_process
#include "Properties.inc"
#include "TargetProperties.inc"

enum {
#define LLDB_PROPERTIES_process
#include "PropertiesEnum.inc"
#include "TargetPropertiesEnum.inc"
};

ProcessProperties::ProcessProperties(lldb_private::Process *process)
Expand Down
8 changes: 4 additions & 4 deletions lldb/source/Target/Target.cpp
Expand Up @@ -3274,11 +3274,11 @@ static constexpr OptionEnumValueElement g_memory_module_load_level_values[] = {
"this setting loads sections and all symbols."} };

#define LLDB_PROPERTIES_target
#include "Properties.inc"
#include "TargetProperties.inc"

enum {
#define LLDB_PROPERTIES_target
#include "PropertiesEnum.inc"
#include "TargetPropertiesEnum.inc"
ePropertyExperimental,
};

Expand Down Expand Up @@ -3355,11 +3355,11 @@ class TargetOptionValueProperties : public OptionValueProperties {

// TargetProperties
#define LLDB_PROPERTIES_experimental
#include "Properties.inc"
#include "TargetProperties.inc"

enum {
#define LLDB_PROPERTIES_experimental
#include "PropertiesEnum.inc"
#include "TargetPropertiesEnum.inc"
};

class TargetExperimentalOptionValueProperties : public OptionValueProperties {
Expand Down
File renamed without changes.
4 changes: 2 additions & 2 deletions lldb/source/Target/Thread.cpp
Expand Up @@ -64,11 +64,11 @@ const ThreadPropertiesSP &Thread::GetGlobalProperties() {
}

#define LLDB_PROPERTIES_thread
#include "Properties.inc"
#include "TargetProperties.inc"

enum {
#define LLDB_PROPERTIES_thread
#include "PropertiesEnum.inc"
#include "TargetPropertiesEnum.inc"
};

class ThreadOptionValueProperties : public OptionValueProperties {
Expand Down

0 comments on commit 6a253d3

Please sign in to comment.