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