intrinsified python_cext PySequence_XXX

Author: tomasstupka

graalpython/com.oracle.graal.python.cext/src/abstract.c

@@ -549,7 +549,7 @@ PyObject* PySequence_Concat(PyObject *s, PyObject *o) {
 
 UPCALL_ID(PySequence_InPlaceRepeat);
 PyObject* PySequence_InPlaceRepeat(PyObject *o, Py_ssize_t count) {
-	return UPCALL_CEXT_O(_jls_PySequence_Repeat, native_to_java(o), count);
+	return UPCALL_CEXT_O(_jls_PySequence_InPlaceRepeat, native_to_java(o), count);
 }
 
 UPCALL_ID(PySequence_InPlaceConcat);

graalpython/com.oracle.graal.python.test/src/tests/cpyext/test_abstract.py

@@ -139,6 +139,15 @@ def _reference_fast(args):
         return obj
     return list(obj)
 
+def _wrap_slice_fun(fun, since=0, default=None):
+    def wrapped_fun(args):
+        if not isinstance(args[0], list) and not isinstance(args[0], set) and not isinstance(args[0], tuple) and not isinstance(args[0], str):
+            if sys.version_info.minor >= since:
+                raise SystemError("expected list type")
+            else:
+                return default
+        return fun(args)
+    return wrapped_fun
 
 class NoNumber():
     pass
@@ -733,6 +742,7 @@ def compile_module(self, name):
             ({'a':0, 'b':1},),
             (DummySequence(),),
             (DummyListSubclass(),),
+            ('hello',),
         ),
         resultspec="i",
         argspec='O',
@@ -750,6 +760,7 @@ def compile_module(self, name):
             ([None],),
             (set(),),
             (DummyListSubclass(),),
+            ('hello',),
         ),
         resultspec="n",
         argspec='O',
@@ -772,13 +783,58 @@ def compile_module(self, name):
             (set(), 0),
             ({'a', 'b'}, 0),
             (DummyListSubclass(), 1),
+            ('hello', 1),
         ),
         resultspec="O",
         argspec='On',
         arguments=["PyObject* sequence", "Py_ssize_t idx"],
         cmpfunc=unhandled_error_compare
     )
-
+    
+    test_PySequence_GetSlice = CPyExtFunction(
+        _wrap_slice_fun(lambda args: args[0][args[1]:args[2]]),
+        lambda: (
+            (tuple(), 0, 1),
+            ((1, 2, 3), 1, 2),
+            ((None,), 1, 2),
+            ([], 0, 1),
+            (['a', 'b', 'c'], 1, 2),
+            ([None], 0, 1),
+            (set(), 0, 1),
+            ({'a', 'b'}, 1, 2),
+            (DummyListSubclass(), 0, 1),
+            ('hello', 0, 1),
+        ),
+        resultspec="O",
+        argspec='Onn',
+        arguments=["PyObject* sequence", "Py_ssize_t ilow", "Py_ssize_t ihigh"],
+        cmpfunc=unhandled_error_compare
+    )
+    
+    test_PySequence_Contains = CPyExtFunction(
+        lambda args: args[1] in args[0],
+        lambda: (
+            (tuple(), 1),
+            ((1, 2, 3), 1),
+            ((1, 2, 3), 4),
+            ((None,), 1),
+            ([], 1),
+            (['a', 'b', 'c'], 'a'),
+            (['a', 'b', 'c'], 'd'),
+            ([None], 1),
+            (set(), 1),
+            ({'a', 'b'}, 'a'),
+            ({'a', 'b'}, 'c'),
+            (DummyListSubclass(), 1),
+            ('hello', 'e'),
+            ('hello', 'x'),
+        ),
+        resultspec="i",
+        argspec='OO',
+        arguments=["PyObject* haystack", "PyObject* needle"],
+        cmpfunc=unhandled_error_compare
+    )
+    
     test_PySequence_ITEM = CPyExtFunction(
         _reference_getitem,
         lambda: (
@@ -788,6 +844,7 @@ def compile_module(self, name):
             ([], 10),
             (['a', 'b', 'c'], 2),
             ([None], 0),
+            ('hello', 0),
         ),
         resultspec="O",
         argspec='On',
@@ -803,6 +860,7 @@ def compile_module(self, name):
             ((None,), 1, None),
             ([], 10, 1),
             (['a', 'b', 'c'], 2, 'z'),
+            ('hello', 2, 'z'),
         ),
         code=''' PyObject* wrap_PySequence_SetItem(PyObject* sequence, Py_ssize_t idx, PyObject* value) {
             if (PySequence_SetItem(sequence, idx, value) < 0) {
@@ -830,6 +888,27 @@ def compile_module(self, name):
             ({'a': 0, 'b': 1, 'c': 2},),
             (None,),
             (0,),
+            ('hello',),
+        ),
+        resultspec="O",
+        argspec='O',
+        arguments=["PyObject* sequence"],
+        cmpfunc=unhandled_error_compare
+    )
+    
+    test_PySequence_List = CPyExtFunction(
+        lambda args: list(args[0]),
+        lambda: (
+            (list(),),
+            ((1, 2, 3),),
+            ((None,),),
+            ([],),
+            (['a', 'b', 'c'],),
+            ({'a', 'b', 'c'},),
+            ({'a': 0, 'b': 1, 'c': 2},),
+            (None,),
+            (0,),
+            ('hello',),
         ),
         resultspec="O",
         argspec='O',