RFR: 8212879: Make JVMTI TagMap table concurrent [v3]

[Coleen Phillimore]

On Wed, 4 Nov 2020 07:42:36 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>>> Ok, so I'm not sure what to do with this:
>>> 
>>> enum Phase {
>>> // Serial phase.
>>> JVMTI_ONLY(jvmti)
>>> // Additional implicit phase values follow for oopstorages.
>>> `};`
>>> 
>>> I've removed the only thing in this enum.
>> 
>> Enums without any named enumerators are still meaningful types.  More so with scoped enums, but still with unscoped enums.
>
>> > Though it might be possible to go even further and eliminate WeakProcessorPhases as a thing separate from OopStorageSet.
>> 
>> This makes sense. Can we file another RFE for this? I was sort of surprised by how much code was involved so I tried to find a place to stop deleting.
> 
> I think the deletion stopped at the wrong place; it either went too far, or not far enough.

I restored the empty enum for Phase and can open a new RFE if there is more code to remove.

-------------

PR: https://git.openjdk.java.net/jdk/pull/967