On Sat, 9 May 2026 06:11:59 GMT, Alan Bateman <[email protected]> wrote:
>> @mgronlun how important `jdk.jfr.internal.JVM::getClassId`? >> >> While looking on external addresses referenced in C2 intrinsics we noticed >> that JFR's `getClassId` is not generated: >> >> 1214 456 b 4 jdk.jfr.jvm.TestJFRIntrinsic::getClassIdIntrinsic >> (22 bytes) >> @ 1 jdk.jfr.internal.JVM::getClassId (0 >> bytes) failed to inline: native method >> >> `_getClassId` is missing in LibraryCallKit::try_to_inline() and >> C2Compiler::is_intrinsic_supported(). Even so we have >> `LibraryCallKit::inline_native_classID()`. > >> Hi @vnkozlov , it was removed as part of: >> >> [8284161: Implementation of Virtual Threads >> (Preview)](https://bugs.openjdk.org/browse/JDK-8284161) >> >> Now, if I can only remember why that was the case... > > Seems to be when the intrinsic for _getEventWriter was added in the loom > repo, here's the commit: > > https://github.com/openjdk/loom/commit/d2f8c47523fb10a09815aef33db51c292a77ab97 Thanks @AlanBateman, @egahlin, @merykitty and @vnkozlov for reviews and input on this. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30922#issuecomment-4554196087
