On Mon, 15 Sep 2025 16:00:41 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> Hi all,
>> 
>> `FilteredJavaFieldStream` purely exists to hide the field `constantPoolOop` 
>> in `jdk.internal.reflect.ConstantPool`. This PR:
>> 1) refactors `constantPoolOop` to be a hidden VM injected field `vmholder` 
>> (name to be consistent with other similar cases),
>> 2) removes `FilteredJavaFieldStream` entirely, and
>> 3) updates code previously relying on `FilteredJavaFieldStream` to use 
>> `JavaFieldStream` instead.
>> 
>> Testing: Oracle tiers 1-3.
>
> ConstantPool in the JVM metadata used to be called constantPoolOop, so I 
> think that's what it's looking to not find.

Thanks for the reviews, @coleenp @liach @jdksjolen @alexmenkov. I've merged in 
the latest mainline which had a merge commit.

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

PR Comment: https://git.openjdk.org/jdk/pull/27209#issuecomment-3297070955

Reply via email to