On Fri, 17 May 2024 06:03:00 GMT, Adam Sotona <[email protected]> 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.
>
> test/jdk/jdk/internal/reflect/CallerSensitive/CheckCSMs.java line 151:
>
>> 149:
>> 150: boolean needsCsm = false;
>> 151: for (var element : code) {
>
> Scanning the instructions is a bit different approach than in the original
> test.
Same as the other method name check, it was manually extracted ReferenceFinder
logic, we should retire ReferenceFinder with the old class file library.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19206#discussion_r1604405594
