Skip to content

Commit 2abdcc9

Browse files
authored
Merge pull request #83040 from augusto2112/sub-remote-addr
[NFC][RemoteInspection] Subtracting remote addresses should return int
2 parents b1eea5e + 3d19b9c commit 2abdcc9

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

include/swift/Remote/RemoteAddress.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -117,10 +117,10 @@ class RemoteAddress {
117117
return RemoteAddress(Data - rhs, getAddressSpace());
118118
}
119119

120-
RemoteAddress operator-(const RemoteAddress &rhs) const {
121-
if (AddressSpace != rhs.AddressSpace)
122-
return RemoteAddress();
123-
return RemoteAddress(Data - rhs.Data, getAddressSpace());
120+
uint64_t operator-(const RemoteAddress &rhs) const {
121+
assert(AddressSpace == rhs.AddressSpace &&
122+
"Comparing remote addresses of different address spaces");
123+
return Data - rhs.Data;
124124
}
125125

126126
template <typename IntegerType>

include/swift/RemoteInspection/TypeRefBuilder.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ class ReflectionSection {
7777
RemoteRef<U> getRemoteRef(remote::RemoteAddress remoteAddr) const {
7878
assert(containsRemoteAddress(remoteAddr, sizeof(U)));
7979
auto localAddr = (uint64_t)(uintptr_t)Start.getLocalBuffer() +
80-
(remoteAddr - Start.getRemoteAddress()).getRawAddress();
80+
(remoteAddr - Start.getRemoteAddress());
8181

8282
return RemoteRef<U>(remoteAddr, (const U *)localAddr);
8383
}

0 commit comments

Comments
 (0)