Skip to content

Commit 5cd027d

Browse files
committed
FunctionNode#getClosure: use System.arraycopy
1 parent 79378f2 commit 5cd027d

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/BuiltinConstructors.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -136,9 +136,9 @@
136136
import com.oracle.graal.python.builtins.objects.common.HashingStorageLibrary;
137137
import com.oracle.graal.python.builtins.objects.common.SequenceNodes;
138138
import com.oracle.graal.python.builtins.objects.common.SequenceNodes.GetObjectArrayNode;
139+
import com.oracle.graal.python.builtins.objects.common.SequenceNodesFactory.GetObjectArrayNodeGen;
139140
import com.oracle.graal.python.builtins.objects.common.SequenceStorageNodes;
140141
import com.oracle.graal.python.builtins.objects.common.SequenceStorageNodes.NoGeneralizationNode;
141-
import com.oracle.graal.python.builtins.objects.common.SequenceNodesFactory.GetObjectArrayNodeGen;
142142
import com.oracle.graal.python.builtins.objects.complex.PComplex;
143143
import com.oracle.graal.python.builtins.objects.dict.PDict;
144144
import com.oracle.graal.python.builtins.objects.ellipsis.PEllipsis;
@@ -2186,12 +2186,8 @@ public PFunction function(Object cls, PCode code, PDict globals, String name, PT
21862186
}
21872187

21882188
private static PCell[] getClosure(Object[] closure) {
2189-
assert closure != null;
21902189
PCell[] cells = new PCell[closure.length];
2191-
for (int i = 0; i < closure.length; i++) {
2192-
assert closure[i] instanceof PCell;
2193-
cells[i] = (PCell) closure[i];
2194-
}
2190+
PythonUtils.arraycopy(closure, 0, cells, 0, closure.length);
21952191
return cells;
21962192
}
21972193

0 commit comments

Comments
 (0)