RFR: 8345959: Make JVM_IsStaticallyLinked JVM_LEAF

David Holmes dholmes at openjdk.org
Thu Dec 12 06:45:40 UTC 2024


On Wed, 11 Dec 2024 23:44:46 GMT, Jiangli Zhou <jiangli at openjdk.org> wrote:

>> This is likely fine. I must admit I am not aware of the differences between JVM_ENTRY_NO_ENV and JVM_LEAF, and just picked one that seemed reasonable to me given context and other usages.
>
> Thanks for the quick review, @magicus!
> 
> I didn't catch this details while reviewing https://github.com/openjdk/jdk/commit/a136a85b6f5bbc92727883693c1ce31c37a82fd5 either.

@jianglizhou please remember the two reviewer rule for hotspot changes and the 24 hour rule for non-trivial changes. When Magnus admits to not knowing the significance of leaf versus non-leaf his one review hardly seems sufficient in any case.

The change to leaf is of course fine.

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

PR Comment: https://git.openjdk.org/jdk/pull/22685#issuecomment-2537944214


More information about the hotspot-dev mailing list