[Lldb-commits] [lldb] 12606d1 - Revert "Expose GetAddressingBits() in the Process API."
Adrian Prantl via lldb-commits
lldb-commits at lists.llvm.org
Wed Mar 23 15:28:42 PDT 2022
Author: Adrian Prantl
Date: 2022-03-23T15:28:34-07:00
New Revision: 12606d16bc3e1c75eefc781863b199b930137068
URL: https://github.com/llvm/llvm-project/commit/12606d16bc3e1c75eefc781863b199b930137068
DIFF: https://github.com/llvm/llvm-project/commit/12606d16bc3e1c75eefc781863b199b930137068.diff
LOG: Revert "Expose GetAddressingBits() in the Process API."
This reverts commit 7504dd5e00f514628614db8ee07514c73220e597.
In newer review feedback it was pointed out that there is a better API for this in Process::GetCodeAddressMask().
Added:
Modified:
lldb/include/lldb/Target/Process.h
lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h
Removed:
################################################################################
diff --git a/lldb/include/lldb/Target/Process.h b/lldb/include/lldb/Target/Process.h
index 7c4302342988b..7d8fbb7797d89 100644
--- a/lldb/include/lldb/Target/Process.h
+++ b/lldb/include/lldb/Target/Process.h
@@ -699,9 +699,6 @@ class Process : public std::enable_shared_from_this<Process>,
/// Get the system architecture for this process.
virtual ArchSpec GetSystemArchitecture() { return {}; }
- /// Return the number of bits in a vmaddr that are used by valid addresses.
- virtual llvm::Optional<uint32_t> GetAddressingBits() { return {}; }
-
/// Get the system runtime plug-in for this process.
///
/// \return
diff --git a/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp b/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
index 12747321a2932..2e652e11ea110 100644
--- a/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
+++ b/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
@@ -212,10 +212,6 @@ ArchSpec ProcessGDBRemote::GetSystemArchitecture() {
return m_gdb_comm.GetHostArchitecture();
}
-llvm::Optional<uint32_t> ProcessGDBRemote::GetAddressingBits() {
- return m_gdb_comm.GetAddressingBits();
-}
-
bool ProcessGDBRemote::CanDebug(lldb::TargetSP target_sp,
bool plugin_specified_by_name) {
if (plugin_specified_by_name)
diff --git a/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h b/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h
index a227e99b35768..50cef8e499dcc 100644
--- a/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h
+++ b/lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h
@@ -72,8 +72,6 @@ class ProcessGDBRemote : public Process,
ArchSpec GetSystemArchitecture() override;
- llvm::Optional<uint32_t> GetAddressingBits() override;
-
// Check if a given Process
bool CanDebug(lldb::TargetSP target_sp,
bool plugin_specified_by_name) override;
More information about the lldb-commits
mailing list