On Wed, 25 Mar 2026 03:41:07 GMT, David Holmes <[email protected]> wrote:
>> Matthias Baesken has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Move JVM_dladdr to jvm_md.h > > src/hotspot/os/posix/include/jvm_md.h line 66: > >> 64: >> 65: #if defined(AIX) >> 66: #include "jni_md.h" > > The current header file is only included in `jvm.h` which first includes > `jni.h` so I think we can rely on that and avoid the need to include > `jni_md.h` directly. Correction sorry - I see a number of platform specific files include jvm_md.h directly (which seems unfortunate to say the least). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30295#discussion_r2985587320
