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