RFR (S) 8242365: Shenandoah: use uint16_t instead of jushort for liveness cache

Roman Kennke rkennke at redhat.com
Wed Apr 8 11:37:37 UTC 2020


Looks good.

Thanks,
Roman


> RFE:
>   https://bugs.openjdk.java.net/browse/JDK-8242365
> 
> I was actually looking into this weird statement:
>   size_t max = (1 << (sizeof(jushort) * 8)) - 1;
> 
> ...which is borderline UB, because we can shift for more than datatype width. Then I figured we
> might as well replace jushort with uint16_t.
> 
> Webrev:
>   https://cr.openjdk.java.net/~shade/8242365/webrev.01/
> 
> Testing: hotspot_gc_shenandoah
> 



More information about the shenandoah-dev mailing list