[lworld] RFR: 8377327: [lworld] SA tests fail with --enable-preview

[Coleen Phillimore]

On Fri, 6 Feb 2026 23:08:45 GMT, Coleen Phillimore <coleenp at openjdk.org> 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