RFR: 8338745: Intrinsify Continuation.pin() and Continuation.unpin() [v2]
Markus Grönlund
mgronlun at openjdk.org
Thu Aug 22 11:13:17 UTC 2024
> Greetings,
>
> Please help review this change set that implements C2 intrinsics for jdk.internal.vm.Continuation.pin() and jdk.internal.vm.Continuation.unpin().
>
> This work is a consequence of [JDK-8338417](https://bugs.openjdk.org/browse/JDK-8338417), which required us to introduce explicit pin constructs for Virtual threads in a relatively performance-sensitive path.
>
> Testing: jdk_jfr, loom testing
>
> Comment: I changed the type of the ContinuationEntry::_pin_count field from uint to uin32_t to make the size explicit and to access it uniformly from the intrinsic code using T_INT.
>
> Thanks
> Markus
Markus Grönlund has updated the pull request incrementally with one additional commit since the last revision:
Deoptimization::Action_none for no deopt
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/20664/files
- new: https://git.openjdk.org/jdk/pull/20664/files/ab3af53a..fa9a737a
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=20664&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=20664&range=00-01
Stats: 20 lines in 2 files changed: 1 ins; 13 del; 6 mod
Patch: https://git.openjdk.org/jdk/pull/20664.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/20664/head:pull/20664
PR: https://git.openjdk.org/jdk/pull/20664
More information about the hotspot-compiler-dev
mailing list