RFR: 8365858: FilteredJavaFieldStream is unnecessary
Johan Sjölen
jsjolen at openjdk.org
Mon Sep 15 17:44:39 UTC 2025
On Thu, 11 Sep 2025 08:35:05 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.
Looks good to me!
-------------
Marked as reviewed by jsjolen (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/27209#pullrequestreview-3225668786
More information about the hotspot-runtime-dev
mailing list