Make it transparent for external projects using PyTuple macros to build on GraalPy/PyPy

Author: timfel

newsfragments/1.changed.md

@@ -1 +1,2 @@
 Enable vectorcall methods on GraalPy
+Provide compatibility shims in PyTuple macros for PyPy and GraalPy

pyo3-ffi/src/cpython/tupleobject.rs

@@ -1,6 +1,8 @@
 use crate::object::*;
 #[cfg(not(PyPy))]
 use crate::pyport::Py_ssize_t;
+#[cfg(any(GraalPy, PyPy))]
+use crate::{PyTuple_GetItem, PyTuple_SetItem};
 
 #[repr(C)]
 pub struct PyTupleObject {
@@ -21,16 +23,26 @@ pub unsafe fn PyTuple_GET_SIZE(op: *mut PyObject) -> Py_ssize_t {
 }
 
 #[inline]
-#[cfg(not(any(PyPy, GraalPy)))]
 pub unsafe fn PyTuple_GET_ITEM(op: *mut PyObject, i: Py_ssize_t) -> *mut PyObject {
-    *(*(op as *mut PyTupleObject)).ob_item.as_ptr().offset(i)
+    #[cfg(not(any(GraalPy, PyPy)))]
+    return *(*(op as *mut PyTupleObject)).ob_item.as_ptr().offset(i);
+    #[cfg(any(GraalPy, PyPy))]
+    return PyTuple_GetItem(op, i);
 }
 
 /// Macro, *only* to be used to fill in brand new tuples
 #[inline]
-#[cfg(not(any(PyPy, GraalPy)))]
 pub unsafe fn PyTuple_SET_ITEM(op: *mut PyObject, i: Py_ssize_t, v: *mut PyObject) {
-    *(*(op as *mut PyTupleObject)).ob_item.as_mut_ptr().offset(i) = v;
+    #[cfg(not(any(GraalPy, PyPy)))]
+    {
+        *(*(op as *mut PyTupleObject)).ob_item.as_mut_ptr().offset(i) = v;
+    }
+    #[cfg(any(GraalPy, PyPy))]
+    {
+        let previous = PyTuple_GetItem(op, i);
+        Py_XINCREF(previous);
+        PyTuple_SetItem(op, i, v);
+    }
 }
 
 // skipped _PyTuple_DebugMallocStats