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