RFR: 8365858: FilteredJavaFieldStream is unnecessary [v2]

[Paul Hübner]

On Tue, 16 Sep 2025 09:35:02 GMT, Paul Hübner <phubner at 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.

Thanks for the reviews everyone.

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

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