On Wed, 19 Apr 2023 16:00:18 GMT, Doug Simon <dnsi...@openjdk.org> wrote:
>> Doug Simon has updated the pull request incrementally with two additional >> commits since the last revision: >> >> - added breadcrumb in AnnotationParser about considering JVMCI should new >> annotation element types be added >> - fixed javadoc comment > > Thanks for the reviews @turbanoff , @vnkozlov and @jddarcy. @dougxc I see next 2 JVMCI tests failed when run with `-XX:TypeProfileLevel=222` on all platforms (our stress testing): compiler/jvmci/jdk.vm.ci.runtime.test/src/jdk/vm/ci/runtime/test/TestResolvedJavaMethod.java compiler/jvmci/jdk.vm.ci.runtime.test/src/jdk/vm/ci/runtime/test/TestResolvedJavaType.java # Internal Error (/workspace/open/src/hotspot/cpu/x86/macroAssembler_x86.cpp:829), pid=2430194, tid=2430218 # fatal error: DEBUG MESSAGE: exact klass and actual klass differ Current thread (0x00007f9ff8460480): JavaThread "MainThread" [_thread_in_Java, id=2430218, stack(0x00007f9fe06b7000,0x00007f9fe07b8000)] Stack: [0x00007f9fe06b7000,0x00007f9fe07b8000], sp=0x00007f9fe07b44e0, free space=1013k Native frames: (J=compiled Java code, j=interpreted, Vv=VM code, C=native code) V [libjvm.so+0x12b8d35] MacroAssembler::debug64(char*, long, long*)+0x45 (macroAssembler_x86.cpp:829) J 1935 c1 sun.reflect.annotation.AnnotationParser.parseAnnotation2(Ljava/nio/ByteBuffer;Ljdk/internal/reflect/ConstantPool;Ljava/lang/Class;Z[Ljava/lang/Class;)Ljava/lang/annotation/Annotation; java.base@21-internal (275 bytes) @ 0x00007f9fe10d4cc6 [0x00007f9fe10d4740+0x0000000000000586] [error occurred during error reporting (printing native stack (with source info)) ------------- PR Comment: https://git.openjdk.org/jdk/pull/12810#issuecomment-1515565983
