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

Aleksey Shipilev shade at openjdk.java.net
Tue May 10 15:08:13 UTC 2022


On Tue, 10 May 2022 15:02:29 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
>
> Markus Grönlund has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - missing comma
>  - riscv

RISC-V runs more or less fine with this patch.

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

Marked as reviewed by shade (Reviewer).

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


More information about the hotspot-jfr-dev mailing list