RFR: 8144874: Reimplement TraceClassResolution with UL

[Coleen Phillimore]

This looks good to me.

Coleen


On 12/17/15 3:00 PM, Max Ockner wrote:
> New webrev:  http://cr.openjdk.java.net/~mockner/classresolve
>
> On 12/16/2015 6:27 PM, Ioi Lam wrote:
>> Hi Max,
>>
>> [1] I think you need to rebase your changes and send out a new 
>> webrev. The latest version of argument.cpp already has these
>>
>> static AliasedFlag const aliased_jvm_logging_flags[] = {
>>   { "-XX:+TraceMonitorInflation", "-Xlog:monitorinflation=debug" },
>>   { "-XX:-TraceMonitorInflation", "-Xlog:monitorinflation=off" },
>>   { NULL, NULL }
>> };
>
> Fixed.
>
>>
>> which would conflict with similar lines that you added.
>>
>> [2] In the test case ClassResolutionTest.java:
>>
>> Is there any need for adding "-Xmx64m"? If not, I think it should be 
>> removed.
>
> Removed.
>
>>
>> [3] Also, could you send out a sample log that covers all of the 
>> different logging lines that you have touched?
>
> What would you like to see? The webrev contains all of the lines that 
> were touched, and there is only one level of logging here 
> (-Xlog:classresolve=info)
>
>>
>> I think the "RESOLVE" in the following output is redundant and should 
>> be removed.
>>
>> [classresolve] RESOLVE 
>> ClassResolutionTest$ClassResolutionTestMain$Thing1Handler 
>> ClassResolutionTest$ClassResolutionTestMain$Thing1
>
> I agree. This is now gone.
>
>>
>> Thanks
>> - Ioi
>>
>> On 12/16/15 12:44 PM, Max Ockner wrote:
>>> Hello all,
>>> Please review my code for another Unified Logging conversion.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8144874
>>> Webrev: http://cr.openjdk.java.net/~mockner/classresolve/
>>>
>>> Summary: "-XX:+TraceClassResolution" reimplemented using Unified 
>>> Logging with classresolve tag and Info level. Support is maintained 
>>> for TraceClassResolution using the alias table.
>>>
>>> Tested with: Selection Resolution tests, jtreg tests. This change 
>>> also adds a jtreg test for the implementation of classresolve, and 
>>> for the maintained support of TraceClassResolution.
>>>
>>> Thanks,
>>> Max
>>
>