[lworld] RFR: 8366093: [lworld] Add preview mode to C++ classloader

[David Beaumont]

On Thu, 9 Oct 2025 19:51:54 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> ModulesReference are different flags, with different semantics. That's why I made sure they were private and provided helper methods for reading/writing the package directory offsets (because I did trip over using the wrong constants at one point).
>> 
>> I agree it's weird/annoying to have this similar-but-different set of flags, but the ModuleReference flags are completely encapsulated in one place, so cannot be accidentally confused with, or misused in place of the ImageLocation flags.
>> 
>> These flags have a requirement of being zero for almost all entries to reduce image file size, while the ModulesReference flags need to additive for merging (and are not stored in the same part of the jimage file).
>> 
>> I could change flag ordering to make the values of the "has preview version" flags match, but since these sets of flags must never be mistaken for each other, I'm not sure that's beneficial (if anything it might foster the idea that they can be used interchangeably in some way).
>
> If they have the same spelling, they will cause confusion if they have different semantics and usages.
> All of imageFile, jimage, and the file system provider and classloader are in a single scope of operation.

I've done this now (but I don't really agree with the argument that it will be likely to cause confusion however, since the flags are stored in different data structures and their use is carefully encapsulated).

-------------

PR Review Comment: https://git.openjdk.org/valhalla/pull/1618#discussion_r2417850049