Don't eagerly create native storages when slicing

Author: msimacek

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/common/SequenceStorageNodes.java

@@ -592,29 +592,25 @@ protected static SequenceStorage doManagedStorage(BasicSequenceStorage storage,
         }
 
         @Specialization
-        protected static NativeSequenceStorage doNativeByte(NativeByteSequenceStorage storage, int start, @SuppressWarnings("unused") int stop, int step, int length,
-                        @Bind("this") Node inliningTarget,
-                        @Cached CStructAccess.ReadByteNode readNode,
-                        @Shared @Cached StorageToNativeNode storageToNativeNode) {
+        protected static SequenceStorage doNativeByte(NativeByteSequenceStorage storage, int start, @SuppressWarnings("unused") int stop, int step, int length,
+                        @Cached CStructAccess.ReadByteNode readNode) {
 
             byte[] newArray = new byte[length];
             for (int i = start, j = 0; j < length; i += step, j++) {
                 newArray[j] = readNode.readArrayElement(storage.getPtr(), i);
             }
-            return storageToNativeNode.execute(inliningTarget, newArray, length);
+            return new ByteSequenceStorage(newArray);
         }
 
         @Specialization
-        protected static NativeSequenceStorage doNativeObject(NativeObjectSequenceStorage storage, int start, @SuppressWarnings("unused") int stop, int step, int length,
-                        @Bind("this") Node inliningTarget,
+        protected static SequenceStorage doNativeObject(NativeObjectSequenceStorage storage, int start, @SuppressWarnings("unused") int stop, int step, int length,
                         @Cached CStructAccess.ReadPointerNode readNode,
-                        @Shared @Cached StorageToNativeNode storageToNativeNode,
                         @Cached NativeToPythonNode toJavaNode) {
             Object[] newArray = new Object[length];
             for (int i = start, j = 0; j < length; i += step, j++) {
                 newArray[j] = toJavaNode.execute(readNode.readArrayElement(storage.getPtr(), i));
             }
-            return storageToNativeNode.execute(inliningTarget, newArray, length);
+            return new ObjectSequenceStorage(newArray);
         }
 
         @NeverDefault