8202377: Modularize C2 GC barriers
Erik Österlund
erik.osterlund at oracle.com
Thu May 17 13:48:45 UTC 2018
Hi Roland,
Thank you for having a look at these changes.
On 2018-05-17 14:58, Roland Westrelin wrote:
> Hi Erik,
>
>> http://cr.openjdk.java.net/~eosterlund/8202377/webrev.01/
> Overall, this looks good to me.
>
> I suppose:
>
> loop_optimize_gc_barrier()
> find_dominating_barriers()
>
> are there for ZGC and they don't feel generic enough to me to be
> included in this API. That is if I were to add another GC or let say
> change the G1 implementation so it's better suited for optimizations, I
> think it's quite unlikely I would reuse either hook. It would be better
> to remove them in my opinion.
Yes they are indeed hooks used only by ZGC at the moment, and I agree
they might be an over-generalization at the moment. I am okay with
removing those hooks now, and then we see what we might have in common
with Shenandoah later when we have integrated them both.
Incremental webrev:
http://cr.openjdk.java.net/~eosterlund/8202377/webrev.01_02/
Full webrev:
http://cr.openjdk.java.net/~eosterlund/8202377/webrev.02/
Again, thanks a lot for looking at these changes.
Thanks,
/Erik
> Roland.
More information about the hotspot-compiler-dev
mailing list