[aarch64-port-dev ] RFR 8233401: Shenandoah: Refactor/cleanup Shenandoah load barrier code

Andrew Haley aph at redhat.com
Tue Nov 5 08:52:20 UTC 2019


On 11/4/19 6:23 PM, Zhengyu Gu wrote:
>> They are still needed for calling super class's load_at(). Even though, 
>> they are not used there neither.

Aha! Sorry, I missed that.

> Or I should say, they are not used there right now, but may be used in 
> future ...

So add them in the future, surely. All you're doing by passing unused
args is confusing the reader. It definitely succeeded with me...

-- 
Andrew Haley  (he/him)
Java Platform Lead Engineer
Red Hat UK Ltd. <https://www.redhat.com>
https://keybase.io/andrewhaley
EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671




More information about the hotspot-gc-dev mailing list