summaryrefslogtreecommitdiff
path: root/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface
diff options
context:
space:
mode:
authorMed Ismail Bennani <ismail@bennani.ma>2024-08-20 10:47:06 -0700
committerGitHub <noreply@github.com>2024-08-20 10:47:06 -0700
commit356533246aa33db44c75afb2d3eb6e0e08f0d7a6 (patch)
treeb6771ec31906f74ffc68778a12f66b61616f13fb /lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface
parent8056d92bc201906edbb27f4efa1cc80a18359880 (diff)
[lldb] Fix windows debug build after 9d07f43 (#104896)
This patch tries to fix an issue with the windows debug builds where the PDB file for python scripted interfaces cannot be opened since its path length exceed the windows `MAX_PATH` limit: https://github.com/llvm/llvm-project/pull/101672#issuecomment-2289481324 This patch addresses the issue by building all the interfaces as a single library plugin that initiliazes each component as part of its `Initialize` method, instead of building each interface as its own library plugin. This keeps the build artifact path length smaller while respecting the naming convention and without making any exception in the build system. Fixes #104895. Signed-off-by: Med Ismail Bennani <ismail@bennani.ma>
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface')
-rw-r--r--lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/CMakeLists.txt16
-rw-r--r--lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.cpp110
-rw-r--r--lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.h61
3 files changed, 0 insertions, 187 deletions
diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/CMakeLists.txt b/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/CMakeLists.txt
deleted file mode 100644
index ae5e525229c0..000000000000
--- a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/CMakeLists.txt
+++ /dev/null
@@ -1,16 +0,0 @@
-add_lldb_library(lldbPluginScriptInterpreterPythonScriptedPlatformPythonInterface PLUGIN
-
- ScriptedPlatformPythonInterface.cpp
-
- LINK_LIBS
- lldbCore
- lldbHost
- lldbInterpreter
- lldbTarget
- lldbPluginScriptInterpreterPython
- ${Python3_LIBRARIES}
- ${LLDB_LIBEDIT_LIBS}
-
- LINK_COMPONENTS
- Support
- )
diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.cpp b/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.cpp
deleted file mode 100644
index 3586251bd4aa..000000000000
--- a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.cpp
+++ /dev/null
@@ -1,110 +0,0 @@
-//===-- ScriptedPlatformPythonInterface.cpp -------------------------------===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-
-#include "lldb/Core/PluginManager.h"
-#include "lldb/Host/Config.h"
-#include "lldb/Target/ExecutionContext.h"
-#include "lldb/Utility/Log.h"
-#include "lldb/Utility/Status.h"
-#include "lldb/lldb-enumerations.h"
-
-#if LLDB_ENABLE_PYTHON
-
-// clang-format off
-// LLDB Python header must be included first
-#include "../../lldb-python.h"
-//clang-format on
-
-#include "../../SWIGPythonBridge.h"
-#include "../../ScriptInterpreterPythonImpl.h"
-#include "ScriptedPlatformPythonInterface.h"
-
-using namespace lldb;
-using namespace lldb_private;
-using namespace lldb_private::python;
-using Locker = ScriptInterpreterPythonImpl::Locker;
-
-LLDB_PLUGIN_DEFINE_ADV(ScriptedPlatformPythonInterface, ScriptInterpreterPythonScriptedPlatformPythonInterface)
-
-ScriptedPlatformPythonInterface::ScriptedPlatformPythonInterface(
- ScriptInterpreterPythonImpl &interpreter)
- : ScriptedPlatformInterface(), ScriptedPythonInterface(interpreter) {}
-
-llvm::Expected<StructuredData::GenericSP>
-ScriptedPlatformPythonInterface::CreatePluginObject(
- llvm::StringRef class_name, ExecutionContext &exe_ctx,
- StructuredData::DictionarySP args_sp, StructuredData::Generic *script_obj) {
- ExecutionContextRefSP exe_ctx_ref_sp =
- std::make_shared<ExecutionContextRef>(exe_ctx);
- StructuredDataImpl sd_impl(args_sp);
- return ScriptedPythonInterface::CreatePluginObject(class_name, script_obj,
- exe_ctx_ref_sp, sd_impl);
-}
-
-StructuredData::DictionarySP ScriptedPlatformPythonInterface::ListProcesses() {
- Status error;
- StructuredData::DictionarySP dict_sp =
- Dispatch<StructuredData::DictionarySP>("list_processes", error);
-
- if (!dict_sp || !dict_sp->IsValid() || error.Fail()) {
- return ScriptedInterface::ErrorWithMessage<StructuredData::DictionarySP>(
- LLVM_PRETTY_FUNCTION,
- llvm::Twine("Null or invalid object (" +
- llvm::Twine(error.AsCString()) + llvm::Twine(")."))
- .str(),
- error);
- }
-
- return dict_sp;
-}
-
-StructuredData::DictionarySP
-ScriptedPlatformPythonInterface::GetProcessInfo(lldb::pid_t pid) {
- Status error;
- StructuredData::DictionarySP dict_sp =
- Dispatch<StructuredData::DictionarySP>("get_process_info", error, pid);
-
- if (!dict_sp || !dict_sp->IsValid() || error.Fail()) {
- return ScriptedInterface::ErrorWithMessage<StructuredData::DictionarySP>(
- LLVM_PRETTY_FUNCTION,
- llvm::Twine("Null or invalid object (" +
- llvm::Twine(error.AsCString()) + llvm::Twine(")."))
- .str(),
- error);
- }
-
- return dict_sp;
-}
-
-Status ScriptedPlatformPythonInterface::AttachToProcess(
- ProcessAttachInfoSP attach_info) {
- // FIXME: Pass `attach_info` to method call
- return GetStatusFromMethod("attach_to_process");
-}
-
-Status ScriptedPlatformPythonInterface::LaunchProcess(
- ProcessLaunchInfoSP launch_info) {
- // FIXME: Pass `launch_info` to method call
- return GetStatusFromMethod("launch_process");
-}
-
-Status ScriptedPlatformPythonInterface::KillProcess(lldb::pid_t pid) {
- return GetStatusFromMethod("kill_process", pid);
-}
-
-void ScriptedPlatformPythonInterface::Initialize() {
- PluginManager::RegisterPlugin(
- GetPluginNameStatic(), "Mock platform and interact with its processes.",
- CreateInstance, eScriptLanguagePython, {});
-}
-
-void ScriptedPlatformPythonInterface::Terminate() {
- PluginManager::UnregisterPlugin(CreateInstance);
-}
-
-#endif // LLDB_ENABLE_PYTHON
diff --git a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.h b/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.h
deleted file mode 100644
index 01ee40a5a197..000000000000
--- a/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedPlatformPythonInterface/ScriptedPlatformPythonInterface.h
+++ /dev/null
@@ -1,61 +0,0 @@
-//===-- ScriptedPlatformPythonInterface.h -----------------------*- C++ -*-===//
-//
-// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-// See https://llvm.org/LICENSE.txt for license information.
-// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-//
-//===----------------------------------------------------------------------===//
-
-#ifndef LLDB_PLUGINS_SCRIPTINTERPRETER_PYTHON_INTERFACES_SCRIPTEDPLATFORMPYTHONINTERFACE_H
-#define LLDB_PLUGINS_SCRIPTINTERPRETER_PYTHON_INTERFACES_SCRIPTEDPLATFORMPYTHONINTERFACE_H
-
-#include "lldb/Host/Config.h"
-#include "lldb/Interpreter/Interfaces/ScriptedPlatformInterface.h"
-
-#if LLDB_ENABLE_PYTHON
-
-#include "../ScriptedPythonInterface.h"
-
-namespace lldb_private {
-class ScriptedPlatformPythonInterface : public ScriptedPlatformInterface,
- public ScriptedPythonInterface,
- public PluginInterface {
-public:
- ScriptedPlatformPythonInterface(ScriptInterpreterPythonImpl &interpreter);
-
- llvm::Expected<StructuredData::GenericSP>
- CreatePluginObject(const llvm::StringRef class_name,
- ExecutionContext &exe_ctx,
- StructuredData::DictionarySP args_sp,
- StructuredData::Generic *script_obj = nullptr) override;
-
- llvm::SmallVector<llvm::StringLiteral> GetAbstractMethods() const override {
- return llvm::SmallVector<llvm::StringLiteral>(
- {"list_processes", "attach_to_process", "launch_process",
- "kill_process"});
- }
-
- StructuredData::DictionarySP ListProcesses() override;
-
- StructuredData::DictionarySP GetProcessInfo(lldb::pid_t) override;
-
- Status AttachToProcess(lldb::ProcessAttachInfoSP attach_info) override;
-
- Status LaunchProcess(lldb::ProcessLaunchInfoSP launch_info) override;
-
- Status KillProcess(lldb::pid_t pid) override;
-
- static void Initialize();
-
- static void Terminate();
-
- static llvm::StringRef GetPluginNameStatic() {
- return "ScriptedPlatformPythonInterface";
- }
-
- llvm::StringRef GetPluginName() override { return GetPluginNameStatic(); }
-};
-} // namespace lldb_private
-
-#endif // LLDB_ENABLE_PYTHON
-#endif // LLDB_PLUGINS_SCRIPTINTERPRETER_PYTHON_INTERFACES_SCRIPTEDPLATFORMPYTHONINTERFACE_H