8217465: RFR(S): [REDO] - Optimize CodeHeap Analytics

[Schmidt, Lutz]

Thank you, Vladimir,

for reviewing and running all the tests. So the error report I got from the submit repo was unrelated noise. The REDO-patch builds and runs on all "our" platforms (s390x, ppc64, ppc64le, AIX) as well. 

Best Regards,
Lutz

On 30.01.19, 01:19, "Vladimir Kozlov" <vladimir.kozlov at oracle.com> wrote:

    Looks good.
    
    It passed tier1,hstier2 testing. It builds on all our platforms.
    
    Thanks,
    Vladimir
    
    On 1/28/19 5:13 AM, Schmidt, Lutz wrote:
    > Hi all,
    > 
    > may I please request reviews for this REDO of JDK-8217250. The only relevant difference of this REDO is that I moved the
    >    #define USE_BUFFEREDSTREAM
    > line further down. It is now located after all the #include statements.
    > 
    > The changeset is included in our inhouse tests since Jan 23rd with no issues detected. It was submitted to jdk/submit on Jan 25th with one failure reported on windows-x64 (see attachment). I cannot relate the failure to my changes. Could someone please have a look at the logs? If the reported failure is a false positive, here are the bug and webrev links for your reviews:
    > 
    > Bug:    https://bugs.openjdk.java.net/browse/JDK-8217465
    > Webrev: http://cr.openjdk.java.net/~lucy/webrevs/8217465.00/
    > 
    > Thanks a lot!
    > Lutz
    > 
    >   
    >