On Thu, 28 Nov 2024 02:21:55 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> After JEP 479 ([JDK-8339783](https://bugs.openjdk.org/browse/JDK-8339783) >> was integrated, the handling of certain symbol lookup code can be >> simplified. The old code needed to support 32-bit Windows, where names had a >> trailing `@<number>`. When this special case now is removed, some >> streamlining is possible. Specifically these arrays: >> >> #define JNI_ONLOAD_SYMBOLS {"JNI_OnLoad"} >> #define JNI_ONUNLOAD_SYMBOLS {"JNI_OnUnload"} >> #define JVM_ONLOAD_SYMBOLS {"JVM_OnLoad"} >> #define AGENT_ONLOAD_SYMBOLS {"Agent_OnLoad"} >> #define AGENT_ONUNLOAD_SYMBOLS {"Agent_OnUnload"} >> #define AGENT_ONATTACH_SYMBOLS {"Agent_OnAttach"} >> >> are all singletons and so the actual strings can just be inlined directly >> into the code that uses them. >> >> Testing: >> - GHA >> - Tiers 1-4 sanity >> - >> Thanks > > David Holmes has updated the pull request incrementally with two additional > commits since the last revision: > > - Merge branch '8343703-jni-symbol' of github.com:dholmes-ora/jdk into > 8343703-jni-symbol > - Fix indent Looks good. ------------- Marked as reviewed by kbarrett (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/22380#pullrequestreview-2466212066