On Thu, 3 Nov 2022 03:41:26 GMT, Kim Barrett <[email protected]> wrote:
>> Coleen Phillimore has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Remove now-unused function that I missed.
>
> src/hotspot/share/gc/z/zHeap.inline.hpp line 48:
>
>> 46: inline uint32_t ZHeap::hash_oop(uintptr_t addr) const {
>> 47: const uintptr_t offset = ZAddress::offset(addr);
>> 48: return ZHash::address_to_uint32(offset);
>
> I think removal of the call to `ZHash::address_to_uint32` means this file no
> longer needs to include zHash.inline.hpp.
I think you are right Kim.
-------------
PR: https://git.openjdk.org/jdk/pull/10938