On Fri, 6 Feb 2026 23:08:45 GMT, Coleen Phillimore <[email protected]> wrote:
>> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/oops/FieldType.java line >> 53: >> >>> 51: break; // Ok. signature char known >>> 52: default: >>> 53: this.first = 'V'; // It's a value type. >> >> I don't understand the purpose of this line. >> In signature format, `V` means void, not value type. >> And value types are encoded with a L; envelope like all classes. > > Thank you for pointing this out. I thought I saw we were adding V for > flattened fields but I was just hallucinating (like AI). This assert was > caused by the misparsing of field info, so I reverted this change. Also this FieldType class isn't really used. It should be removed in mainline. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2064#discussion_r2776389792
