[16] RFR: 8241065: Shenandoah: remove leftover code after JDK-8231086

Roman Kennke rkennke at redhat.com
Wed Aug 12 20:07:13 UTC 2020


Hi Charlie,

The patch looks fine.

Please also post Shenandoah-related changes to shenandoah-dev.

Next time, it would be useful if you could generate the webrev such
that it includes the metadata (commit-msg, user, etc). That should
happen if you simply commit your change locally (or hg qnew it - that
is what I usually do), and simply invoke webrev without any arguments
from the project root dir.

I can sponsor the change.

Thank you!
Roman

> Hi,
> 
> Please review this change to remove the unused ArrayCopyStoreValMode
> enum. 
> The Bug also mentions removing the write_ref_array_pre_work function
> but it was
> previously removed as part of JDK-8242130 [1].
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8241065
> Webrev: 
> http://cr.openjdk.java.net/~burban/cgracie/JDK-8241065/webrev0.0/
> 
> Sponsor required: Yes
> 
> Test:
>    Built on MacOS {release,fastdebug}
> 
> Thanks,
> Charlie Gracie
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8242130
> 




More information about the hotspot-gc-dev mailing list