RFR: 8359064: Expose reason for marking nmethod non-entrant to JVMCI client [v2]

Doug Simon dnsimon at openjdk.org
Wed Jun 11 19:17:32 UTC 2025


On Wed, 11 Jun 2025 19:01:22 GMT, Cesar Soares Lucas <cslucas at openjdk.org> wrote:

>> src/jdk.internal.vm.ci/share/classes/jdk/vm/ci/code/InstalledCode.java line 152:
>> 
>>> 150:      */
>>> 151:     public void invalidate() {
>>> 152:         invalidate(true, 0);
>> 
>> This assigns `ChangeReason::C1_codepatch` to JVMCI invalidations which does not seem right.
>
> I believe zero is mapped to `UNKNOWN` in the ChangeReason enum, if I'm not very mistaken here?!

Sorry, I was looking at nmethod.hpp in my local source without the changes in this PR. That said, it should probably map to `ChangeReason::JVMCI_invalidate_nmethod` right?
BTW, seems like `ChangeReason::JVMCI_invalidate_nmethod_mirror` is unused and could be deleted.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25706#discussion_r2140896474


More information about the graal-dev mailing list