Skip to content

Commit 279dee7

Browse files
committed
No longer need address argument
1 parent df17ec4 commit 279dee7

8 files changed

+12
-17
lines changed

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,7 @@ uint32_t NativeRegisterContextDBReg::SetHardwareWatchpoint(
248248
return LLDB_INVALID_INDEX32;
249249
}
250250

251-
control_value = MakeWatchControlValue(addr, size, watch_flags);
251+
control_value = MakeWatchControlValue(size, watch_flags);
252252

253253
// Iterate over stored watchpoints and find a free wp_index
254254
wp_index = LLDB_INVALID_INDEX32;

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,8 +89,7 @@ class NativeRegisterContextDBReg
8989
AdjustWatchpoint(const WatchpointDetails &details) = 0;
9090

9191
virtual uint32_t MakeBreakControlValue(size_t size) = 0;
92-
virtual uint32_t MakeWatchControlValue(lldb::addr_t addr, size_t size,
93-
uint32_t watch_flags) = 0;
92+
virtual uint32_t MakeWatchControlValue(size_t size, uint32_t watch_flags) = 0;
9493
virtual uint32_t GetWatchpointSize(uint32_t wp_index) = 0;
9594

9695
virtual llvm::Error ReadHardwareDebugInfo() = 0;

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg_arm.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,8 +70,9 @@ uint32_t NativeRegisterContextDBReg_arm::MakeBreakControlValue(size_t size) {
7070
}
7171
}
7272

73-
uint32_t NativeRegisterContextDBReg_arm::MakeWatchControlValue(
74-
lldb::addr_t addr, size_t size, uint32_t watch_flags) {
73+
uint32_t
74+
NativeRegisterContextDBReg_arm::MakeWatchControlValue(size_t size,
75+
uint32_t watch_flags) {
7576
// We can only watch up to four bytes that follow a 4 byte aligned address
7677
// per watchpoint register pair, so make sure we can properly encode this.
7778
// We assume that the address was 4 byte aligned by AdjustWatchpoint.

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg_arm.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ class NativeRegisterContextDBReg_arm : public NativeRegisterContextDBReg {
2626

2727
uint32_t MakeBreakControlValue(size_t size) override;
2828

29-
uint32_t MakeWatchControlValue(lldb::addr_t addr, size_t size,
30-
uint32_t watch_flags) override;
29+
uint32_t MakeWatchControlValue(size_t size, uint32_t watch_flags) override;
3130

3231
bool ValidateBreakpoint(size_t size,
3332
[[maybe_unused]] lldb::addr_t addr) override {

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg_arm64.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,9 @@ uint32_t NativeRegisterContextDBReg_arm64::MakeBreakControlValue(size_t size) {
7878
return m_hw_dbg_enable_bit | pac_bits | encoded_size;
7979
}
8080

81-
uint32_t NativeRegisterContextDBReg_arm64::MakeWatchControlValue(
82-
lldb::addr_t addr, size_t size, uint32_t watch_flags) {
83-
(void)addr;
84-
81+
uint32_t
82+
NativeRegisterContextDBReg_arm64::MakeWatchControlValue(size_t size,
83+
uint32_t watch_flags) {
8584
// PAC (bits 2:1): 0b10
8685
const uint32_t pac_bits = 2 << 1;
8786

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg_arm64.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ class NativeRegisterContextDBReg_arm64 : public NativeRegisterContextDBReg {
2626

2727
uint32_t MakeBreakControlValue(size_t size) override;
2828

29-
uint32_t MakeWatchControlValue(lldb::addr_t addr, size_t size,
30-
uint32_t watch_flags) override;
29+
uint32_t MakeWatchControlValue(size_t size, uint32_t watch_flags) override;
3130
};
3231

3332
} // namespace lldb_private

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg_loongarch.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,7 @@ NativeRegisterContextDBReg_loongarch::MakeBreakControlValue(size_t size) {
5252
}
5353

5454
uint32_t NativeRegisterContextDBReg_loongarch::MakeWatchControlValue(
55-
lldb::addr_t addr, size_t size, uint32_t watch_flags) {
56-
(void)addr;
55+
size_t size, uint32_t watch_flags) {
5756
// Encoding hardware watchpoint control value.
5857
// Size encoded:
5958
// case 1 : 0b11

lldb/source/Plugins/Process/Utility/NativeRegisterContextDBReg_loongarch.h

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,7 @@ class NativeRegisterContextDBReg_loongarch : public NativeRegisterContextDBReg {
2626

2727
uint32_t MakeBreakControlValue(size_t size) override;
2828

29-
uint32_t MakeWatchControlValue(lldb::addr_t addr, size_t size,
30-
uint32_t watch_flags) override;
29+
uint32_t MakeWatchControlValue(size_t size, uint32_t watch_flags) override;
3130
};
3231

3332
} // namespace lldb_private

0 commit comments

Comments
 (0)