On Thu, 9 Jan 2025 08:40:37 GMT, Adam Sotona <asot...@openjdk.org> wrote:
>> There are no more consumers of ASM library except for hotspot tests. >> This patch moves ASM library from java.base module to the hotspot test >> libraries location and fixes the tests. >> >> Please review. >> >> Thanks, >> Adam > > Adam Sotona has updated the pull request incrementally with one additional > commit since the last revision: > > removed jdk.internal package prefix from asm Thanks for the reviews. I'll update the copyright years and run again tier1 - 4 tests (just to be sure). Jtreg seems to have no problems to use ASM as a declared library. This PR primary purpose is to remove the ASM from java.base. Further improvements of hotspot tests using ASM can follow. However I would prefer to see further conversions of the hotspot tests to use Class-File API instead. Long-term goal is to remove ASM from JDK completely. FYI: Regarding the merge of this PR I've been asked to wait until some downstream projects dependent on jdk.internal.objectweb.asm are fixed. ------------- PR Comment: https://git.openjdk.org/jdk/pull/22946#issuecomment-2582985694