RFR (S) 8241740: Shenandoah: remove ShenandoahHeapRegion::_heap
Roman Kennke
rkennke at redhat.com
Fri Mar 27 13:45:12 UTC 2020
Looks good, thanks!
Roman
> RFE:
> https://bugs.openjdk.java.net/browse/JDK-8241740
>
> It is not seem to worth it to drag the _heap field in every region. Almost all the places where it
> is used is not performance-critical, and ShenandoahHeap::heap() is actually as fast, especially
> after JDK-8241743. Ditching this field saves 8 bytes per region.
>
> I had to move seqnum_last_alloc_mutator to avoid including shenandoahHeap.inline.hpp in a generic
> header.
>
> Webrev:
> https://cr.openjdk.java.net/~shade/8241740/webrev.02/
>
> Testing: hotspot_gc_shenandoah, eyeballing shenandoahHeapRegion.o objdumps
>
More information about the hotspot-gc-dev
mailing list