diff options
| author | David Spickett <david.spickett@linaro.org> | 2022-09-28 14:44:17 +0000 |
|---|---|---|
| committer | David Spickett <david.spickett@linaro.org> | 2022-10-12 08:19:30 +0000 |
| commit | 6faa345da9d747c65c4d901c4ef26dfedf95da45 (patch) | |
| tree | fdf18419d1fb763dbce102011c9e2d6ff2c1aa9d /lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp | |
| parent | 32f0bde548cface29b26ee26763881dbcfb8bb58 (diff) | |
[LLDB] Pass const RegisterInfo& to RegisterValue::SetValueFromData
Familiar story, callers are either checking upfront that the pointer
wasn't null or not checking at all. SetValueFromData itself didn't
check either.
So make the parameter a ref and fixup the few places where a nullptr
check seems needed.
Depends on D135668
Reviewed By: clayborg
Differential Revision: https://reviews.llvm.org/D135670
Diffstat (limited to 'lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp')
| -rw-r--r-- | lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp b/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp index ead8c4b4a80d..84a19d5b1303 100644 --- a/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp +++ b/lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp @@ -81,7 +81,7 @@ bool RegisterContextMemory::ReadRegister(const RegisterInfo *reg_info, } const bool partial_data_ok = false; return reg_value - .SetValueFromData(reg_info, m_reg_data, reg_info->byte_offset, + .SetValueFromData(*reg_info, m_reg_data, reg_info->byte_offset, partial_data_ok) .Success(); } |