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

Cesar Soares Lucas cslucas at openjdk.org
Wed Jun 11 19:04:29 UTC 2025


On Wed, 11 Jun 2025 07:50:43 GMT, Doug Simon <dnsimon at openjdk.org> wrote:

>> Cesar Soares Lucas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address PR feedback: add comments, refactor enum definition in the Java side.
>
> 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?!

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

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


More information about the graal-dev mailing list