RFR: 8328111: Convert Runtime tests to use the ClassFile API instead of ASM [v12]

Oussama Louati duke at openjdk.org
Mon Mar 18 12:22:28 UTC 2024


On Mon, 18 Mar 2024 11:51:52 GMT, Oussama Louati <duke at openjdk.org> wrote:

>> The main goal pull request migrate the existing tests to utilize the new ClassFile API instead of the ASM library. The use of ASM has served us well in the past, but maintaining it has its costs and limitations.
>> 
>> ### **Cost of Maintaining ASM Library:**
>> 
>> - _Risk of Changes:_ The ASM library is an external dependency, and updates or changes in its APIs can introduce unforeseen issues or require significant code modifications in our tests.
>> - _Lack of Reviewers:_ As the ASM library evolves, finding reviewers familiar with its intricacies becomes challenging, leading to delays in reviewing and merging changes.
>> 
>> ### **Reasons for Migration:**
>> 
>> - _Stability and Consistency:_ Utilizing the ClassFile API ensures stability and compatibility with the Java platform, reducing the risk of compatibility issues due to external library changes.
>> - _Sustainability:_ The ClassFile API is a standard part of Java, making it easier to find reviewers and maintain code consistency across test suites.
>> - _Future Compatibility:_ As Java evolves, relying on native Java APIs like the ClassFile API ensures future compatibility and reduces technical debt.
>> 
>> ### **Concerns Addressed:**
>> 
>> - Risk of Changes: By migrating to a standard Java API, we mitigate the risks associated with external library changes and ensure smoother maintenance and updates in the future.
>> - Backporting and Compatibility: The use of native Java APIs allows for easier backporting of test fixes and ensures compatibility across Java versions, including preview releases.
>
> Oussama Louati has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - import ClassDesc
>  - convert the anewarray test to use the ClassFile API

test/hotspot/jtreg/runtime/verifier/TestANewArray.java line 1:

> 1: /*

Test description:
The TestANewArray test validates the JVM's handling of the anewarray bytecode instruction for creating arrays with many dimensions. It creates classes with 254, 255, and 264 dimensions and checks the JVM's responses. 

Test Outcome:
The test should pass for arrays with 254 dimensions and fail for arrays with 255 and 264 dimensions. The failure messages should indicate an array with too many dimensions or an illegal anewarray instruction. The test is successful if these outcomes are met.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18271#discussion_r1528437100


More information about the hotspot-runtime-dev mailing list