RFR: 8365858: FilteredJavaFieldStream is unnecessary
Paul Hübner
phubner at openjdk.org
Tue Sep 16 09:35:03 UTC 2025
On Mon, 15 Sep 2025 16:00:41 GMT, Coleen Phillimore <coleenp 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.
>
> 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
More information about the hotspot-runtime-dev
mailing list