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:

http://cr.openjdk.java.net/~roland/8212243/webrev.03/

Roland.


More information about the hotspot-compiler-dev mailing list