RFR: Remove 5 more develop options from GC code.

[David Lindholm]

Eric,

Thanks for looking at this! I'll update the comment. Do you need a new 
webrev or could I consider this reviewed by you?


Thanks,
David

On 2015-10-21 16:23, Eric Caspole wrote:
>
> Hi David,
> Always great to erase a lot of code. I just have one comment about 
> updating the comments near here since UseAsyncConcMarkSweepGC is gone:
>
> in concurrentMarkSweepGeneration.cpp
>
> 1758     if (_foregroundGCIsActive) {
> 1759       // The foreground collector is active or we're
> 1760       // not using asynchronous collections.  Skip this
> 1761       // background collection.
>
> Regards,
> Eric
>
>
> On 10/21/2015 07:53 AM, David Lindholm wrote:
>> Jesper,
>>
>> Thanks for the review! Could I have one more reviewer looking at this?
>>
>>
>> Thanks,
>> David
>>
>> On 2015-10-20 22:48, Jesper Wilhelmsson wrote:
>>> Looks good!
>>> Thanks for removing flags!!!
>>> /Jesper
>>>
>>> Den 20/10/15 kl. 11:54, skrev David Lindholm:
>>>> Hi,
>>>>
>>>> Please review this patch that removes the following 5 develop options
>>>> from the
>>>> GC code:
>>>>
>>>> UseCMSAdaptiveFreeLists
>>>> UseAsyncConcMarkSweepGC
>>>> CMSDictionaryChoice
>>>> CMSOverflowEarlyRestoration
>>>> CMSTestInFreeList
>>>>
>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8139952
>>>> Webrev: http://cr.openjdk.java.net/~david/JDK-8139952/webrev.00/
>>>>
>>>>
>>>> Thanks,
>>>> David
>>