================
@@ -16,6 +16,16 @@
namespace lldb_private {
+class ARMSingleStepBreakpointLocationsPredictor
+ : public SingleStepBreakpointLocationsPredictor {
+public:
+ ARMSingleStepBreakpointLocationsPredictor(
+ std::unique_ptr<EmulateInstruction> emulator_up)
+ : SingleStepBreakpointLocationsPredictor{std::move(emulator_up)} {}
----------------
DavidSpickett wrote:
```
class SingleStepBreakpointLocationsPredictor {
public:
SingleStepBreakpointLocationsPredictor(
std::unique_ptr<EmulateInstruction> emulator_up)
: m_emulator_up{std::move(emulator_up)} {}
```
Seems like you just pass on to this. Can this sub-class not just use the base
class constructor?
https://github.com/llvm/llvm-project/pull/127505
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
