On Fri, 9 May 2025 22:34:45 GMT, Chen Liang <li...@openjdk.org> wrote:

>> For early eval; test by changing the ClassReader max accepted version of 
>> test ASM to 24 instead of 25
>
> Chen Liang has updated the pull request with a new target base due to a merge 
> or a rebase. The incremental webrev excludes the unrelated changes brought in 
> by the merge/rebase. The pull request contains six additional commits since 
> the last revision:
> 
>  - Move other tier 4,5, etc tests to not use ClassReader
>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
> fix/asm-test-upgrade
>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
> fix/asm-test-upgrade
>  - Use classfile api instead of javac setting version
>  - Merge branch 'master' of https://github.com/openjdk/jdk into 
> fix/asm-test-upgrade
>  - 8356548: Avoid using ASM to parse latest class files in tests

The runtime test that I know looks good.  Ask @iwanowww and @sspitsyn about the 
compiler and redefinition tests.

test/hotspot/jtreg/runtime/MirrorFrame/Asmator.java line 38:

> 36:                     public void atStart(CodeBuilder builder) {
> 37:                         // make receiver go dead!
> 38:                         builder.aconst_null().astore(0);

This looks much better.

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

Marked as reviewed by coleenp (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25124#pullrequestreview-2847613695
PR Review Comment: https://git.openjdk.org/jdk/pull/25124#discussion_r2093670551

Reply via email to