RFR: 8257676: Simplify WeakProcessorPhase
Kim Barrett
kbarrett at openjdk.java.net
Fri Dec 11 07:22:04 UTC 2020
On Wed, 9 Dec 2020 17:20:25 GMT, Ivan Walulya <iwalulya at openjdk.org> wrote:
>> Please review this reimplementation of WeakProcessorPhase. It is changed to
>> a scoped enum at namespace scope, and uses the recently added EnumIterator
>> facility to provide iteration, rather than a bespoke iterator class.
>>
>> This is a step toward eliminating it entirely. I've split it out into a
>> separate PR to make the review of the follow-up work a bit easier.
>>
>> As part of this the file weakProcessorPhases.hpp is renamed to
>> weakProcessorPhase.hpp, but git doesn't seem to be recognizing that as a
>> rename and (majorly) edit, instead treating it as a remove and add a new
>> file.
>>
>> Testing: mach5 tier1
>
> lgtm
Thanks @walulyai , @albertnetymk , @tschatzl for reviews.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1620
More information about the hotspot-gc-dev
mailing list