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

[David Holmes]

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
>>
>>
>