Request for review: JDK-8003581, , UseG1GC is not properly accounted for by INCLUDE_ALTERNATE_GCS

[David Holmes]

On 25/01/2013 4:45 AM, Bob Vandette wrote:
> You claim to be defaulting to SerialGC but you don't set UseG1GC to false?

The UNSUPPORTED_OPTION macro does that.

But I don't see anything forcing use of serialGC. And force_serial_gc() 
should not be altered.

> If you build with INCLUDE_ALL_GCS=1, the G1 code will be available and be used.

Only on those platforms that support it.

David

> Bob.
>
> On Jan 24, 2013, at 1:38 PM, Joe Provino wrote:
>
>> Webrev is here:  http://cr.openjdk.java.net/~jprovino/8003581/webrev.00
>>
>> It's a change to one file:  arguments.cpp.
>>
>> thanks.
>>
>> joe
>