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