Integrated: 8332109: Convert remaining tests using com.sun.tools.classfile to ClassFile API

Chen Liang liach at openjdk.org
Fri May 17 12:27:13 UTC 2024


On Sun, 12 May 2024 08:36:44 GMT, Chen Liang <liach at openjdk.org> wrote:

> Some tests are not migrated to the ClassFile API in previous migrations.
> 
>  - Some are simple oversights that didn't remove usages of com.sun.tools.classfile;
>  - The CallerSensitive ones used an old utility, replaced by CF API-based new code;
>  - many in javac are because the files are compiled with older source compatibility. Those patches are converted to have the source code stored in text blocks and compiled within tests using `ToolBox#writeJavaFiles` and `CompilerUtils.compile`;
>  - As described in the JBS issue, there are a few other tests not covered; one is in #19193 while the others are blocked by CreateSymbols migration or bugs.
> 
> Testing: all modified tests pass.

This pull request has now been integrated.

Changeset: beeffd46
Author:    Chen Liang <liach at openjdk.org>
Committer: Adam Sotona <asotona at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/beeffd4671649e5d8f9c96f0455ac90a82917234
Stats:     1539 lines in 19 files changed: 224 ins; 1098 del; 217 mod

8332109: Convert remaining tests using com.sun.tools.classfile to ClassFile API

Reviewed-by: asotona

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

PR: https://git.openjdk.org/jdk/pull/19206


More information about the compiler-dev mailing list