diff options
author | David Spickett <david.spickett@linaro.org> | 2022-04-13 13:27:46 +0100 |
---|---|---|
committer | David Spickett <david.spickett@linaro.org> | 2022-04-28 14:57:40 +0100 |
commit | b809c4cdb70a100c85524555a14332f22ede1b7a (patch) | |
tree | 9b43e542c4112266c0d4fea8cb19a682f00b1341 /lldb/source/Commands/CommandObjectMemory.cpp | |
parent | 744a8378387e484982d61cf27a4d9598153da8a3 (diff) | |
download | llvm-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.cpp | 2 |
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; |