On Thu, 7 Oct 2021 13:54:16 GMT, Per Liden <pli...@openjdk.org> wrote:
>> Niklas Radomski has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Update autoconf error message
>> - Remove copyright headers
>
> src/hotspot/share/gc/z/zBarrierSetAssembler.cpp line 30:
>
>> 28:
>> 29: Address ZBarrierSetAssemblerBase::address_bad_mask_from_thread(Register
>> thread) {
>> 30: return Address(thread, (intptr_t)
>> ZThreadLocalData::address_bad_mask_offset());
>
> Instead of casting here in this platform agnostic code, I'd suggest that you
> add a new constructor for `Address` on PPC, one that takes `(Register,
> ByteSize)` arguments. Other platforms have that, so I'm a bit surprised that
> PPC doesn't already have that too.
Good catch, thank you for the suggestion! Totally agreed, I changed it
accordingly.
-------------
PR: https://git.openjdk.java.net/jdk/pull/5842
