On Mon, 28 Apr 2025 17:35:13 GMT, Vladimir Kozlov <k...@openjdk.org> wrote:
>> Igor Veresov has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 32 commits: >> >> - Merge branch 'master' into pp2 >> - Fix class filtering >> - Remove the workaround of setting AOTRecordTraining during assembly >> - Address some of the review comments >> - Merge branch 'master' into pp >> - Add AOTCompileEagerly flag to control compilation after clinit >> - Port 8355334: [leyden] Missing type profile info in archived training data >> - Port 8355296: [leyden] Some methods are stuck at level=0 with >> -XX:-TieredCompilation >> - Use ENABLE_IF macro >> - Missing part of the last commit >> - ... and 22 more: https://git.openjdk.org/jdk/compare/2447b981...7fb7ae62 > > Looks better. > There are still places where UL is used specifically for TD processing. > Consider using `(aot, training)` there instead of `(cds)`. Vladimir (@vnkozlov), I did the changes that you requested. You please do another pass? Thanks! ------------- PR Comment: https://git.openjdk.org/jdk/pull/24886#issuecomment-2843621301
