RFR: 8265129: Add intrinsic support for JVM.getClassId [v4]

Denghui Dong ddong at openjdk.java.net
Mon May 10 04:49:02 UTC 2021


On Sat, 8 May 2021 06:52:13 GMT, Denghui Dong <ddong at openjdk.org> wrote:

>> 8265129: Add intrinsic support for JVM.getClassId
>
> Denghui Dong has updated the pull request incrementally with one additional commit since the last revision:
> 
>   swap the positions of two operands in cmp operation since the det register will be modified in 32 bit

Thanks for your comment!

> Does it make sense to intrinsify it in C1?
> IMO, C1 is not easy to get it handled correctly.
>
> How about just intrinsifying it in C2?
> Maybe it will help the review process.
> 
You are right, I found that C1 cannot handle correctly if there are many branches, and that's the reason why I made a platform-dependent code stub in C1.
I agree with you that only intensify this method in C2 now, and I will delete the C1-part.

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

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


More information about the hotspot-compiler-dev mailing list