Immediately load memoryview implementation after C API init.

Author: fangerer

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

@@ -167,9 +167,6 @@ static void initialize_capi() {
     initialize_exceptions();
     initialize_hashes();
     initialize_bufferprocs();
-
-    // we need to make 'memoryview' available
-    PyImport_ImportModule("_memoryview");
 }
 
 __attribute__((always_inline))

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

@@ -142,6 +142,7 @@ public boolean run() {
     @ImportStatic(Message.class)
     public abstract static class CreateDynamic extends PythonBuiltinNode {
         protected static final String INITIALIZE_CAPI = "initialize_capi";
+        protected static final String IMPORT_NATIVE_MEMORYVIEW = "import_native_memoryview";
         private static final String LLVM_LANGUAGE = "llvm";
         @Child private SetItemNode setItemNode;
         @Child private Node isNullNode = Message.IS_NULL.createNode();
@@ -229,6 +230,10 @@ private void ensureCapiWasLoaded() {
                 CallUnaryMethodNode callNode = CallUnaryMethodNode.create();
                 callNode.executeObject(readNode.execute(getContext().getCore().lookupBuiltinModule("python_cext"), INITIALIZE_CAPI), capi);
                 getContext().setCapiWasLoaded();
+
+                // initialization needs to be finished already but load memoryview implemenation
+                // immediately
+                callNode.executeObject(readNode.execute(getContext().getCore().lookupBuiltinModule("python_cext"), IMPORT_NATIVE_MEMORYVIEW), capi);
             }
         }
 

graalpython/lib-graalpython/python_cext.py

@@ -1147,6 +1147,11 @@ def initialize_capi(capi_library):
     initialize_datetime_capi()
 
 
+def import_native_memoryview(capi_library):
+    import _memoryview
+    assert _memoryview is not None
+
+
 def initialize_datetime_capi():
     import datetime