[lworld] RFR: 8357785: [lworld] TestResolvedJavaType fails due to unexpected getInstanceFields order [v3]

Quan Anh Mai qamai at openjdk.org
Fri Jul 25 08:55:17 UTC 2025


On Fri, 25 Jul 2025 08:23:42 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Marc Chevalier has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Second round of comments
>
> Thanks for the thorough analysis Marc! I'm glad to see this code being fixed and cleaned up. Looks all good to me, I just found a few minor things.
> 
>> Nice, I have a pending issue of making this sort less a minefield but removing it completely is absolutely better.
> 
> @merykitty Should we close [JDK-8348547](https://bugs.openjdk.org/browse/JDK-8348547) as duplicate then? If so, I would suggest to integrate the regression tests attached to this bug with this PR. @marc-chevalier you might want to verify that they pass now.

@TobiHartmann Unfortunately, I don't think this PR solves [JDK-8348547](https://bugs.openjdk.org/browse/JDK-8348547). It solves [JDK-8357186](https://bugs.openjdk.org/browse/JDK-8357186), though.

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

PR Comment: https://git.openjdk.org/valhalla/pull/1511#issuecomment-3116954932


More information about the valhalla-dev mailing list