On Thu, 10 Sep 2026 03:36:54 GMT, Serguei Spitsyn <[email protected]> wrote:
>> Yasumasa Suenaga has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Check null marker >> - ValueKlass should be evaluated before InstanceKlass > > src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/oops/OopField.java line > 49: > >> 47: var sig = getSignature().asString(); >> 48: var klsName = sig.substring(1, sig.length() - 1); // extracts >> L(class name); >> 49: return SystemDictionaryHelper.findInstanceKlass(klsName); > > Q: I'm curious what might happen if there are more than one `ClassLoader` > having class with the same name? It feels like this potential issue is more > important for flattened field layouts. I removed this method, and get `ValueKlass` from `ValueFieldLayoutInfo` in new commit, thus it should work even if same name classes are loaded in other class loaders. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/32310#discussion_r3976008976
