RFR: 8277411: C2 fast_unlock intrinsic on AArch64 has unnecessary ownership check

Erik Österlund eosterlund at openjdk.java.net
Fri Nov 26 14:58:09 UTC 2021


On Fri, 26 Nov 2021 13:30:54 GMT, Nils Eliasson <neliasso at openjdk.org> wrote:

>> The AArch64 fast_unlock C2 code checks if the current thread owns the lock. This can be surprisingly expensive in workload where locking is contended. The check is however optional (helpful only for finding JNI code bugs), and indeed not emitted for x86_64. This patch removes the check on AArch64 as well.
>
> Approved!

Thanks for the reviews @neliasso and @nick-arm!

-------------

PR: https://git.openjdk.java.net/jdk/pull/6498


More information about the hotspot-compiler-dev mailing list