On 11/30/17 6:22 AM, Roman Kennke wrote:
Hi David, I added the tag as you proposed: Differential: http://cr.openjdk.java.net/~rkennke/8191564/webrev.08.diff/ Full: http://cr.openjdk.java.net/~rkennke/8191564/webrev.08/
This looks really good. This version looks much better than an early version I looked at (thanks for regenerating webrev with hg rename).
A minor comment that GCMemoryManager could take an enum to indicate the type of this GC action (major vs minor). This can be a future cleanup.
thanks Mandy