RFR(M): 8212243: More gc interface tweaks for arraycopy and some other smaller changes in preparation for Shenandoah

Roland Westrelin rwestrel at redhat.com
Fri Oct 26 09:15:51 UTC 2018

Thanks for re-reviewing this

> I think this 'bs' is not used in arraycopynode.cpp:
> +  BarrierSetC2* bs = BarrierSet::barrier_set()->barrier_set_c2();
>     for (int i = 0; i < count; i++) {
> It seems you wanted to pass it to new load() and store() methods. May be do that instead of loading it inside  these 
> methods especially since in some placed they are called in loop.

Here is a new webrev that implements this suggestion:



More information about the hotspot-compiler-dev mailing list