RFR 8245823: Shenandoah: inline/optimize ShenandoahEvacOOMScope
rkennke at redhat.com
Wed May 27 12:04:49 UTC 2020
> > > Okay, updated:
> > > http://cr.openjdk.java.net/~zgu/JDK-8245823/webrev.01/index.html
> > >
> > > -Zhengyu
> > Looks good in general.
> > + void register_thread_to_protocol(Thread* t);
> > + void unregister_thread_to_protocol(Thread* t);
> > It would have to be 'unregister_thread_from_protocol' but then I
> > think
> > we can just do 'register_thread' and 'unregister_thread' instead?
> Sure, will rename before push.
> > Also, why not inline register_thread() and unregister_thread() too?
> > Those appear to be the common paths in mutator-LRB, and should only
> > account for a few instructions?
> This patch is really intended to address nesting scopes, to shred a
> nano seconds from code root evacuation. I did not study barrier
> so let's experiment that in separate CR, Okay?
Sure, ok then!
More information about the hotspot-gc-dev