RFR: 8301703: java.base jdk.internal.foreign.abi.BindingSpecializer uses ASM to generate classes [v2]
Jorn Vernee
jvernee at openjdk.org
Thu Mar 30 19:55:17 UTC 2023
On Thu, 30 Mar 2023 18:43:42 GMT, Chen Liang <liach at openjdk.org> wrote:
>> Jorn Vernee has updated the pull request incrementally with one additional commit since the last revision:
>>
>> use existing MTD_void constant
>
> src/java.base/share/classes/jdk/internal/foreign/abi/BindingSpecializer.java line 216:
>
>> 214: if (PERFORM_VERIFICATION) {
>> 215: boolean printResults = false; // only print in case of exception
>> 216: CheckClassAdapter.verify(new ClassReader(bytes), null, printResults, new PrintWriter(System.err));
>
> Classfile API provides verification functionality as well, as seen here:
> https://github.com/openjdk/jdk/blob/83cf28f99639d80e62c4031c4c9752460de5f36c/test/jdk/jdk/classfile/ClassHierarchyInfoTest.java#L105
Does this provide additional verification over what is already done just by generating the class?
For instance, IIRC the ASM verifier could catch e.g. stack underflow, but that seems to be caught already by the new implementation without running the verifier.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13247#discussion_r1153722648
More information about the core-libs-dev
mailing list