> 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
