[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