On Wed, 21 Jan 2026 15:18:50 GMT, Matias Saavedra Silva <[email protected]> 
wrote:

>> The alternate substitutability method relies on an injected Java object 
>> "acmp_maps" which is currently not being archived and thus leads to crashes 
>> attempting to run with a CDS archive. The issue stems from inline klasses 
>> being archived and thus loaded at dumptime while the acmp_maps oops is 
>> generated in the class file parser, leading this oop to be absent at 
>> runtime. 
>> 
>> Additionally, the other injected static field "null_reset" was not being 
>> archived either, so both of these fields are properly stored in the archived 
>> heap. In the case of CDS/AOT configurations where the heap is not dumped, 
>> acmp_maps is regenerated at class loading using a copy of the array stored 
>> in metadata.
>> 
>> Tests and APIs are updated to conform to the new output generated by the use 
>> of acmp_maps and remove some test cases which target the old 
>> substitutability method.
>
> Matias Saavedra Silva has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Fixed copyright

test/jdk/ProblemList-AotJdk.txt line 47:

> 45: java/util/Locale/UseOldISOCodesTest.java                       0000000 
> generic-all
> 46: 
> 47: valhalla/valuetypes/SubstitutabilityTest.java                  8374025 
> generic-all

@matias9927 - Why did this PR remove this entry? The bugID for this PR is 
8374729, but
the entry is for 8374025 and has introduced faillures in Tier5.

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

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/1903#discussion_r2717626253

Reply via email to