RFR: 8356548: Avoid using ASM to parse latest class files in tests

Coleen Phillimore coleenp at openjdk.org
Thu May 8 18:35:54 UTC 2025


On Thu, 8 May 2025 14:57:05 GMT, Chen Liang <liach at openjdk.org> wrote:

> For early eval; test by changing the ClassReader max accepted version of test ASM to 24 instead of 25

test/hotspot/jtreg/compiler/calls/common/InvokeDynamicPatcher.java line 76:

> 74:             throw new Error("TESTBUG: Can't get code source" + ex, ex);
> 75:         }
> 76:         try (FileInputStream fis = new FileInputStream(filePath.toFile())) {

Don't you have to delete lines 149-155 also?

test/hotspot/jtreg/runtime/MirrorFrame/Test8003720.java line 30:

> 28:  * @library /testlibrary/asm
> 29:  * @modules java.base/jdk.internal.misc
> 30:  * @compile -XDignore.symbol.file -source 21 -target 21 Victim.java

Why is this necessary?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25124#discussion_r2080231780
PR Review Comment: https://git.openjdk.org/jdk/pull/25124#discussion_r2080233032


More information about the serviceability-dev mailing list