Avoid virtual invoke in 'Array.copyOf' in case of empty kwargs.

fangerer · fangerer · commit a5d61e65c23c · 2018-09-05T14:32:04.000+02:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/argument/ReadVarKeywordsNode.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/argument/ReadVarKeywordsNode.java
@@ -108,7 +108,11 @@ Object extractKwargs(VirtualFrame frame,
                 i++;
             }
         }
-        return returnValue(Arrays.copyOf(remArguments, i));
+        if (remArguments.length != i) {
+            return returnValue(Arrays.copyOf(remArguments, i));
+        } else {
+            return returnValue(remArguments);
+        }
     }
 
     @Specialization(replaces = "extractKwargs")

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,11 @@ Object extractKwargs(VirtualFrame frame,`
`108`	`108`	`i++;`
`109`	`109`	`}`
`110`	`110`	`}`
`111`		`- return returnValue(Arrays.copyOf(remArguments, i));`
	`111`	`+ if (remArguments.length != i) {`
	`112`	`+ return returnValue(Arrays.copyOf(remArguments, i));`
	`113`	`+ } else {`
	`114`	`+ return returnValue(remArguments);`
	`115`	`+ }`
`112`	`116`	`}`
`113`	`117`
`114`	`118`	`@Specialization(replaces = "extractKwargs")`