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

[Joe Provino]

Would anyone have time to review this updated change.  It's one file -- 
arguments.cpp.

thanks.

joe

On 1/25/2013 11:25 AM, Joe Provino wrote:
> 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
>>>