RFR: 8294977: Convert test/jdk/java tests from ASM library to Classfile API [v11]
Adam Sotona
asotona at openjdk.org
Wed Jan 17 08:32:59 UTC 2024
On Sun, 17 Dec 2023 23:11:10 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Summaries:
>> 1. A few recommendations about updating the constant API is made at https://mail.openjdk.org/pipermail/classfile-api-dev/2023-March/000233.html and I may update this patch shall the API changes be integrated before
>> 2. One ASM library-specific test, `LambdaAsm` is removed. Others have their code generation infrastructure upgraded from ASM to Classfile API.
>> 3. Most tests are included in tier1, but some are not:
>> In `:jdk_io`: (tier2, part 2)
>>
>> test/jdk/java/io/Serializable/records/SerialPersistentFieldsTest.java
>> test/jdk/java/io/Serializable/records/ProhibitedMethods.java
>> test/jdk/java/io/Serializable/records/BadCanonicalCtrTest.java
>>
>> In `:jdk_instrument`: (tier 3)
>>
>> test/jdk/java/lang/instrument/RetransformAgent.java
>> test/jdk/java/lang/instrument/NativeMethodPrefixAgent.java
>> test/jdk/java/lang/instrument/asmlib/Instrumentor.java
>>
>>
>> @asotona Would you mind reviewing?
>
> Chen Liang has updated the pull request incrementally with one additional commit since the last revision:
>
> Fix the 2 failing tests and add notes
Yes, CodeBuilder method renames will require some refactoring.
This is good.
-------------
Marked as reviewed by asotona (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/13009#pullrequestreview-1826743185
More information about the core-libs-dev
mailing list