aboutsummaryrefslogtreecommitdiff
path: root/lldb/source/Commands/CommandObjectMemory.cpp
diff options
context:
space:
mode:
authorDavid Spickett <david.spickett@linaro.org>2022-04-13 13:27:46 +0100
committerDavid Spickett <david.spickett@linaro.org>2022-04-28 14:57:40 +0100
commitb809c4cdb70a100c85524555a14332f22ede1b7a (patch)
tree9b43e542c4112266c0d4fea8cb19a682f00b1341 /lldb/source/Commands/CommandObjectMemory.cpp
parent744a8378387e484982d61cf27a4d9598153da8a3 (diff)
downloadllvm-b809c4cdb70a100c85524555a14332f22ede1b7a.zip
llvm-b809c4cdb70a100c85524555a14332f22ede1b7a.tar.gz
llvm-b809c4cdb70a100c85524555a14332f22ede1b7a.tar.bz2
[lldb] Add FixAnyAddress to ABI plugins
FixAnyAddress is to be used when we don't know or don't care whether we're fixing a code or data address. By using FixAnyAddress over the others, you document that no specific choice was made. On all existing platforms apart from Arm Thumb, you could use either FixCodeAddress or FixDataAddress and be fine. Up until now I've chosen to use FixDataAddress but if I had chosen to use FixCodeAddress that would have broken Arm Thumb. Hence FixAnyAddress, to give you the "safest" option when you're in generic code. Uses of FixDataAddress in memory region code have been changed to FixAnyAddress. The functionality is unchanged. Reviewed By: omjavaid, JDevlieghere Differential Revision: https://reviews.llvm.org/D124000
Diffstat (limited to 'lldb/source/Commands/CommandObjectMemory.cpp')
-rw-r--r--lldb/source/Commands/CommandObjectMemory.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/lldb/source/Commands/CommandObjectMemory.cpp b/lldb/source/Commands/CommandObjectMemory.cpp
index 4e63802..e94306c 100644
--- a/lldb/source/Commands/CommandObjectMemory.cpp
+++ b/lldb/source/Commands/CommandObjectMemory.cpp
@@ -1688,7 +1688,7 @@ protected:
// address size, etc.), the end of mappable memory will be lower
// than that. So if we find any non-address bit set, we must be
// at the end of the mappable range.
- (abi && (abi->FixDataAddress(load_addr) != load_addr))) {
+ (abi && (abi->FixAnyAddress(load_addr) != load_addr))) {
result.AppendErrorWithFormat("'%s' takes one argument:\nUsage: %s\n",
m_cmd_name.c_str(), m_cmd_syntax.c_str());
return false;