[riscv-port] RFR: 8277877: C2 fast_unlock intrinsic on riscv has unnecessary ownership check
It's an optional, maybe only for debug, to check the owner of the lock in the fast_unlock intrinsic in riscv, which already applied in AArch64. ------------- Commit messages: - 8277877: C2 fast_unlock intrinsic on riscv has unnecessary ownership check Changes: https://git.openjdk.java.net/riscv-port/pull/14/files Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=14&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8277877 Stats: 5 lines in 1 file changed: 1 ins; 4 del; 0 mod Patch: https://git.openjdk.java.net/riscv-port/pull/14.diff Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/14/head:pull/14 PR: https://git.openjdk.java.net/riscv-port/pull/14
On Sat, 27 Nov 2021 03:27:00 GMT, Yadong Wang <yadongwang@openjdk.org> wrote:
It's an optional, maybe only for debug, to check the owner of the lock in the fast_unlock intrinsic in riscv, which already applied in AArch64.
Looks good. ------------- Marked as reviewed by fyang (Lead). PR: https://git.openjdk.java.net/riscv-port/pull/14
On Sat, 27 Nov 2021 03:27:00 GMT, Yadong Wang <yadongwang@openjdk.org> wrote:
It's an optional, maybe only for debug, to check the owner of the lock in the fast_unlock intrinsic in riscv, which already applied in AArch64.
This pull request has now been integrated. Changeset: 9fef2cab Author: Yadong Wang <yadongwang@openjdk.org> Committer: Fei Yang <fyang@openjdk.org> URL: https://git.openjdk.java.net/riscv-port/commit/9fef2cab2e64de910e5da14093b60... Stats: 5 lines in 1 file changed: 1 ins; 4 del; 0 mod 8277877: C2 fast_unlock intrinsic on riscv has unnecessary ownership check Reviewed-by: fyang ------------- PR: https://git.openjdk.java.net/riscv-port/pull/14
participants (2)
-
Fei Yang
-
Yadong Wang