[Lldb-commits] [PATCH] D135670: [LLDB] Pass const RegisterInfo& to RegisterValue::SetValueFromData
David Spickett via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Oct 11 07:08:13 PDT 2022
DavidSpickett created this revision.
Herald added a project: All.
DavidSpickett requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
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 <https://reviews.llvm.org/D135668>
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D135670
Files:
lldb/include/lldb/Utility/RegisterValue.h
lldb/source/Core/ValueObjectRegister.cpp
lldb/source/Core/ValueObjectVariable.cpp
lldb/source/Plugins/ABI/AArch64/ABIMacOSX_arm64.cpp
lldb/source/Plugins/ABI/AArch64/ABISysV_arm64.cpp
lldb/source/Plugins/Process/Utility/RegisterContextMemory.cpp
lldb/source/Plugins/Process/gdb-remote/GDBRemoteRegisterContext.cpp
lldb/source/Utility/RegisterValue.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D135670.466806.patch
Type: text/x-patch
Size: 9844 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20221011/e7d15921/attachment-0001.bin>
More information about the lldb-commits
mailing list