RFR(S): 8217658 baseline_cleanups from Async Monitor Deflation project

[Daniel D. Daugherty]

David,

Thanks for the re-review. Still hoping for one more reviewer...

Dan


On 1/25/19 9:46 PM, David Holmes wrote:
> Thanks Dan, incremental changes look good.
>
> David
>
> On 26/01/2019 3:38 am, Daniel D. Daugherty wrote:
>> Greetings,
>>
>> I've updated the patch to include the two additional cleanups from 
>> David H.
>> I've also rebased the patch to the jdk/jdk repo at the 'jdk-13+5' tag;
>> there were no merge surprises.
>>
>> Here's an incremental webrev:
>>
>> http://cr.openjdk.java.net/~dcubed/8217658-webrev/1-for-jdk13.inc/
>>
>> I've also generated a full webrev since I still need one more 
>> reviewer for this:
>>
>> http://cr.openjdk.java.net/~dcubed/8217658-webrev/1-for-jdk13.full/
>>
>> I've also updated the bug to talk about the two additional cleanups.
>>
>> Thanks, in advance, for any questions, comments or suggestions.
>>
>> Dan
>>
>>
>> On 1/23/19 3:07 PM, Daniel D. Daugherty wrote:
>>> Greetings,
>>>
>>> I have a (S)mall patch extracted from the Async Monitor Deflation
>>> project that is ready for code review.
>>>
>>> The short version of what this patch is about:
>>>
>>>     This sub-task captures updates to the baseline code that are not
>>>     directly related to Async Monitor Deflation.
>>>
>>> The details are in the bug report:
>>>
>>>     JDK-8217658 baseline_cleanups from Async Monitor Deflation project
>>>     https://bugs.openjdk.java.net/browse/JDK-8217658
>>>
>>> Here's the webrev:
>>>
>>> http://cr.openjdk.java.net/~dcubed/8217658-webrev/0-for-jdk13/
>>>
>>> This patch along with the other two patches for Async Monitor Deflation
>>> project have been through Mach5 
>>> builds-tier1,hs-tier1,jdk-tier1,hs-tier2,hs-tier3
>>> testing and I'm currently running my stress testing kits on my 
>>> Linux-X64
>>> and Solaris-X64 servers.
>>>
>>> Thanks, in advance, for any questions, comments or suggestions.
>>>
>>> Dan
>>>
>>>
>>