[11u] RFR(S): 8250635 backport: MethodArityHistogram should use Compile_lock in favour of fancy checks

[Schmidt, Lutz]

Tests did not reveal any issues. From this end, downport is ready to be pushed. Bug was updated accordingly.
Thanks,
Lutz


On 14.04.21, 11:21, "Doerr, Martin" <martin.doerr at sap.com> wrote:

    Yes. Just let me know when it's ready.

    Best regards,
    Martin


    > -----Original Message-----
    > From: Schmidt, Lutz <lutz.schmidt at sap.com>
    > Sent: Mittwoch, 14. April 2021 11:18
    > To: Doerr, Martin <martin.doerr at sap.com>; hotspot-compiler-
    > dev at openjdk.java.net; jdk-updates-dev at openjdk.java.net; Hohensee,
    > Paul <hohensee at amazon.com>
    > Subject: Re: [11u] RFR(S): 8250635 backport: MethodArityHistogram should
    > use Compile_lock in favour of fancy checks
    > 
    > Thank you, Martin and Paul, for your reviews.
    > 
    > Submit will have to wait another day. I was too late for testing last night.
    > 
    > I will need a sponsor anyway. Is it ok to contact you, Martin, once the tests
    > are done successfully?
    > 
    > Thanks,
    > Lutz
    > 
    > On 14.04.21, 11:06, "Doerr, Martin" <martin.doerr at sap.com> wrote:
    > 
    >     Hi Lutz,
    > 
    >     your backport looks good.
    > 
    >     Best regards,
    >     Martin
    > 
    > 
    >     > -----Original Message-----
    >     > From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> On
    >     > Behalf Of Schmidt, Lutz
    >     > Sent: Dienstag, 13. April 2021 22:02
    >     > To: hotspot-compiler-dev at openjdk.java.net; jdk-updates-
    >     > dev at openjdk.java.net
    >     > Subject: Re: [11u] RFR(S): 8250635 backport: MethodArityHistogram
    > should
    >     > use Compile_lock in favour of fancy checks
    >     >
    >     > Sorry for spamming, forgot jdk-updates-dev.
    >     > Lutz
    >     >
    >     > On 13.04.21, 21:43, "Schmidt, Lutz" <lutz.schmidt at sap.com> wrote:
    >     >
    >     >     Dear Community,
    >     >
    >     >     I would appreciate receiving reviews for this downport change. It is a
    > small
    >     > change in one file only. Unfortunately, it did not apply clean.
    >     >
    >     >     Original bug:    https://bugs.openjdk.java.net/browse/JDK-8250635
    >     >     Downport webrev:
    >     > https://cr.openjdk.java.net/~lucy/webrevs/8250635.11u.01/
    >     >
    >     >     Tests:
    >     >     SAP's internal build and test farm. Results pending.
    >     >
    >     >     Thank you!
    >     >     Lutz
    >     >
    >     >
    >