Skip to content

Commit acef5a2

Browse files
committed
Fix specialization order dependency in storage_to_native
1 parent 3ad7e37 commit acef5a2

File tree

1 file changed

+1
-9
lines changed

1 file changed

+1
-9
lines changed

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

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -825,14 +825,6 @@ private void validate(HashingStorage dictStorage) {
825825
@Builtin(name = "storage_to_native", minNumOfPositionalArgs = 1)
826826
@GenerateNodeFactory
827827
abstract static class StorageToNative extends PythonUnaryBuiltinNode {
828-
@Specialization
829-
@TruffleBoundary
830-
Object toNative(PBytesLike bytes) {
831-
CApiContext.ensureCapiWasLoaded();
832-
NativeSequenceStorage newStorage = ToNativeStorageNode.executeUncached(bytes.getSequenceStorage(), true);
833-
bytes.setSequenceStorage(newStorage);
834-
return bytes;
835-
}
836828

837829
@Specialization
838830
@TruffleBoundary
@@ -847,7 +839,7 @@ Object toNative(PArray array) {
847839
@TruffleBoundary
848840
Object toNative(PSequence sequence) {
849841
CApiContext.ensureCapiWasLoaded();
850-
NativeSequenceStorage newStorage = ToNativeStorageNode.executeUncached(sequence.getSequenceStorage(), false);
842+
NativeSequenceStorage newStorage = ToNativeStorageNode.executeUncached(sequence.getSequenceStorage(), sequence instanceof PBytesLike);
851843
sequence.setSequenceStorage(newStorage);
852844
return sequence;
853845
}

0 commit comments

Comments
 (0)