On Tue, 16 Sep 2025 09:35:02 GMT, Paul Hübner <phub...@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.
>
> Paul Hübner has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains seven commits:
> 
>  - Merge branch 'master' into JDK-8365858
>  - Merge branch 'master' into JDK-8365858
>  - Fix typo.
>  - Refactor to purely use injected fields.
>  - Remove FilteredJavaFieldStream and usages.
>  - Remove constantPoolOop from Java code.
>  - Make constantPoolOop a hidden vmholder field.

Marked as reviewed by jsjolen (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/27209#pullrequestreview-3229698749

Reply via email to