RFR (S) 8244510: Shenandoah: invert SHC2Support::is_in_cset condition

Roman Kennke rkennke at redhat.com
Wed May 6 09:57:35 UTC 2020


Looks reasonable.

Thank you,
Roman

> RFE:
>   https://bugs.openjdk.java.net/browse/JDK-8244510
> 
> Current code in SHC2Support::is_in_cset treats "in cset" branch as frequent. We should really invert
> it after SFX barriers make that condition much less likely. It improves some benchmarks without any
> observable regressions on others. Note I opted to invert the condition instead of changing the
> probabilities, because I think it flows better: test_in_cset is UNLIKELY to be true now.
> 
> Webrev:
>   https://cr.openjdk.java.net/~shade/8244510/webrev.01/
> 
> Testing: hotspot_gc_shenandoah, benchmarks
> 



More information about the shenandoah-dev mailing list