[lworld] RFR: 8357785: [lworld] TestResolvedJavaType fails due to unexpected getInstanceFields order [v2]
Marc Chevalier
mchevalier at openjdk.org
Tue Jul 22 07:49:33 UTC 2025
On Sun, 20 Jul 2025 07:40:44 GMT, Quan Anh Mai <qamai at openjdk.org> wrote:
>> Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Address Quan Anh's comments
>
> src/hotspot/share/oops/fieldStreams.hpp line 325:
>
>> 323: * Doesn't traverse interfaces for now (let's decide how when/if the needs appear).
>> 324: */
>> 325: template<typename FieldStreamType>
>
> By "doesn't traverse interfaces for now" I assume this traverses both static and non-static fields. Tbh I think traversing static fields seems questionable, or maybe I miss some cases, maybe this stream can discard those, too?
I guess if we are not going through the interfaces, indeed, it wouldn't be reasonable to traverse half the static fields.
> test/hotspot/jtreg/ProblemList.txt line 135:
>
>> 133: runtime/valhalla/inlinetypes/verifier/StrictInstanceFieldsTest.java CODETOOLS-7904031 generic-all
>> 134: runtime/valhalla/inlinetypes/verifier/StrictStaticFieldsTest.java CODETOOLS-7904031 generic-all
>> 135: runtime/cds/appcds/RewriteBytecodesInlineTest.java 8361082 generic-all
>
> This issue is fixed with #1498 , so this seems like a merge conflict, right?
Exactly.
-------------
PR Review Comment: https://git.openjdk.org/valhalla/pull/1511#discussion_r2221542613
PR Review Comment: https://git.openjdk.org/valhalla/pull/1511#discussion_r2221543892
More information about the valhalla-dev
mailing list