8340587: Optimize StackMapGenerator$Frame::checkAssignableTo

Reviewed-by: liach
openjdk · Sep 25, 2024 · 2d38af6 · 2d38af6 · openjdk-notifier · Sep 25, 2024
1 parent 9bcc7b6
commit 2d38af6
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/java.base/share/classes/jdk/internal/classfile/impl/StackMapGenerator.java b/src/java.base/share/classes/jdk/internal/classfile/impl/StackMapGenerator.java
@@ -1095,11 +1095,15 @@ void copyFrom(Frame src) {
         }
 
         void checkAssignableTo(Frame target) {
+            int localsSize = this.localsSize;
+            int stackSize = this.stackSize;
             if (target.flags == -1) {
-                target.locals = locals == null ? null : Arrays.copyOf(locals, localsSize);
+                target.locals = locals == null ? null : locals.clone();
                 target.localsSize = localsSize;
-                target.stack = stack == null ? null : Arrays.copyOf(stack, stackSize);
-                target.stackSize = stackSize;
+                if (stackSize > 0) {
+                    target.stack = stack.clone();
+                    target.stackSize = stackSize;
+                }
                 target.flags = flags;
                 target.dirty = true;
             } else {