RFR: Eliminate extra forwarding pointer per object
Roman Kennke
rkennke at redhat.com
Wed May 8 15:27:08 UTC 2019
> On 5/8/19 5:03 PM, Roman Kennke wrote:
>> http://cr.openjdk.java.net/~rkennke/eliminate-fwdptr/webrev.03/
>
> Looks good. I think it is good to go to sh/jdk, and we would follow-up on any subsequent issues later.
>
> Minor nit in shenandoahHeap.inline.hpp, need to fix indent a bit:
>
> 445 assert (cs >= tams, "only objects past TAMS here: " PTR_FORMAT " (" PTR_FORMAT ")",
> p2i(cs), p2i(tams));
> 446 assert (cs < limit, "only objects below limit here: " PTR_FORMAT " (" PTR_FORMAT ")",
> p2i(cs), p2i(limit));
>
> -Aleksey
>
Good spot. Updated:
http://cr.openjdk.java.net/~rkennke/eliminate-fwdptr/webrev.04/
If you're ok, I will cherry-pick JDK-8223567 to sh/jdk and push this
patch on top of it.
Roman
More information about the shenandoah-dev
mailing list