[Lldb-commits] [PATCH] D17395: Refactor GetSoftwareBreakpointTrapOpcode
Aidan Dodds via lldb-commits
lldb-commits at lists.llvm.org
Thu Feb 18 10:02:43 PST 2016
ADodds created this revision.
ADodds added reviewers: clayborg, emaste, ted, zturner, jasonmolenda.
ADodds added a subscriber: lldb-commits.
ADodds set the repository for this revision to rL LLVM.
Herald added a subscriber: emaste.
This patch aims to reduce the code duplication among all of the platforms in GetSoftwareBreakpointTrapOpcode. Common code has been pushed into the Platform base class, and only special case platform code has been left in the derived platforms.
When remote debugging it can be the case that your current host platform will be queried for the trap opcode to use on the target, which currently lead to different results depending on the host lldb is running on. With this patch, platforms have a more unified view of trap opcodes for targets.
Some platforms however have specific/special case requirements so I should not have effected their functionality.
Do others think this is a usefull refactor, or have suggestions/feedback?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 17783 bytes
Desc: not available
More information about the lldb-commits