RFR: 8334898: Resolve static field/method references at CDS dump time [v3]

[Andrew Dinn]

On Fri, 24 Oct 2025 18:39:19 GMT, Ashutosh Mehra <asmehra at openjdk.org> wrote:

>> This patch pre-resolves constant pool entries referred by getstatic, putstatic and invokestatic bytecodes in the assembly phase.
>> It also extends ResolvedConstants.java test to run in AOT mode workflow and additional test for checking resolution of static CP entries.
>> 
>> Before this PR, stats on pre-resolved CP entries in the assembly phase reported by -Xlog:aot=info:
>> 
>> [2.161s][info ][aot        ] Class  CP entries =  12553, archived =   3707 ( 29.5%), reverted =      0
>> [2.161s][info ][aot        ] Field  CP entries =   5236, archived =   1355 ( 25.9%), reverted =      0
>> [2.161s][info ][aot        ] Method CP entries =   3835, archived =   3818 ( 99.6%), reverted =     17
>> [2.161s][info ][aot        ] Indy   CP entries =      9, archived =      9 (100.0%), reverted =      0
>> 
>> 
>> After this PR:
>> 
>> 2.323s][info ][aot        ] Class  CP entries =  12553, archived =   3700 ( 29.5%), reverted =      0
>> [2.323s][info ][aot        ] Field  CP entries =   5236, archived =   3519 ( 67.2%), reverted =      0
>> [2.323s][info ][aot        ] Method CP entries =  21027, archived =   5527 ( 26.3%), reverted =     17
>> [2.323s][info ][aot        ] Indy   CP entries =    353, archived =      9 (  2.5%), reverted =      0
>> 
>> 
>> Update: The stats quoted above are for a simple HelloWorld program
>
> Ashutosh Mehra has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Move check for VM_Version::supports_fast_class_init_checks() to
>   remove_resolved_method_entries_if_non_deterministic()
>   
>   Signed-off-by: Ashutosh Mehra <asmehra at redhat.com>

Marked as reviewed by adinn (Reviewer).

Sorry, I thought I had already ticked the approve box.

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

PR Review: https://git.openjdk.org/jdk/pull/27958#pullrequestreview-3387599592
PR Comment: https://git.openjdk.org/jdk/pull/27958#issuecomment-3455406668