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

[Joe Provino]

With changes from Bob and David, here is a new webrev:

http://cr.openjdk.java.net/~jprovino/8003581/webrev.01

joe

On 1/25/2013 12:44 AM, David Holmes wrote:
> 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
>>