RFR: 8256813: Simplify WeakProcessor counting of OopStorage entries
Stefan Karlsson
stefank at openjdk.java.net
Mon Nov 23 08:35:58 UTC 2020
On Sat, 21 Nov 2020 05:08:57 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
> Please review this change to WeakProcessor's counting of weak and dead
> entries. After JDK-8212879 we no longer need different closure classes to
> deal with the serial and oopstorage cases, since the serial case no longer
> exists. We can collapse that down to one closure class that is tailored for
> the remaining uses.
>
> Also deleted a stale comment left over from JDK-8212879.
>
> Testing:
> mach5 tier1
> Locally (linux-x64) hotspot:tier1 with Serial and Parallel GCs.
Looks good, but would like to get StefanJ's question resolved.
src/hotspot/share/gc/shared/weakProcessor.hpp line 74:
> 72:
> 73: private:
> 74: template<typename IsAlive, typename KeepAlive> class CountingClosure;
May I suggest breaking this into two lines? It's too easy to start to read this as:
template<typename ...>
class GangTask;
-------------
Marked as reviewed by stefank (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/1362
More information about the hotspot-gc-dev
mailing list