RFR: 8286480: Remove the c1 getEventWriter() intrinsic to simplify post-Loom integration platform-porting efforts

Coleen Phillimore coleenp at openjdk.java.net
Tue May 10 14:21:49 UTC 2022


On Tue, 10 May 2022 14:01:24 GMT, Markus Grönlund <mgronlun at openjdk.org> wrote:

> Greetings,
> 
> The C1 intrinsic for getEventWriter() did not materialize and is currently only bouncing a platform-specific stub to enter the VM, putting an unnecessary burden on post-Loom integration platform-porting efforts.
> 
> The platform-specific stub for C2, jfr_write_checkpoint, is still needed but is only called if a java/lang/VirtualThread is mounted. As such, it could be left unimplemented.
> 
> Testing: jdk_jfr
> 
> Thanks
> Markus

This looks good but might want to time it with the x86 32 and ppc loom changes, where they've added stubs for these already.

-------------

Marked as reviewed by coleenp (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/8631


More information about the hotspot-jfr-dev mailing list