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