[GR-11200] TruffleBoundary and some additions to cext

PullRequest: graalpython/147

Author: lukasstadler

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

@@ -49,3 +49,12 @@ PyThreadState * PyThreadState_Get() {
     // TODO: (tfel) how much ThreadState will we actually support?
     return (PyThreadState*)PyThreadState_GetDict();
 }
+
+PyGILState_STATE PyGILState_Ensure() {
+    // ignore for the time being
+    return NULL;
+}
+
+void PyGILState_Release(PyGILState_STATE state) {
+    // ignore for the time being
+}

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

@@ -163,6 +163,7 @@ PArray arrayWithSequenceInitializer(PythonClass cls, String typeCode, PSequence
         }
 
         @Specialization
+        @TruffleBoundary
         PArray arrayWithObjectInitializer(@SuppressWarnings("unused") PythonClass cls, @SuppressWarnings("unused") String typeCode, Object initializer) {
             throw new RuntimeException("Unsupported initializer " + initializer);
         }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/PythonObjectNativeWrapperMR.java

@@ -477,12 +477,17 @@ Object doGeneric(Object object, String key) {
             if (object instanceof PythonAbstractObject) {
                 PythonObjectNativeWrapper nativeWrapper = ((PythonAbstractObject) object).getNativeWrapper();
                 assert nativeWrapper != null;
-                PythonLanguage.getLogger().log(Level.FINE, "read of Python struct native member " + key);
+                logGeneric(key);
                 return getGetItemNode().execute(nativeWrapper.getNativeMemberStore(), key);
             }
             throw UnknownIdentifierException.raise(key);
         }
 
+        @TruffleBoundary(allowInlining = true)
+        private static void logGeneric(String key) {
+            PythonLanguage.getLogger().log(Level.FINE, "read of Python struct native member " + key);
+        }
+
         protected boolean eq(String expected, String actual) {
             return expected.equals(actual);
         }
@@ -643,13 +648,18 @@ Object doGeneric(Object object, String key, Object value) {
             if (object instanceof PythonAbstractObject) {
                 PythonObjectNativeWrapper nativeWrapper = ((PythonAbstractObject) object).getNativeWrapper();
                 assert nativeWrapper != null;
-                PythonLanguage.getLogger().log(Level.FINE, "write of Python struct native member " + key);
+                logGeneric(key);
                 getSetItemNode().execute(nativeWrapper.createNativeMemberStore(), key, value);
                 return value;
             }
             throw UnknownIdentifierException.raise(key);
         }
 
+        @TruffleBoundary(allowInlining = true)
+        private static void logGeneric(String key) {
+            PythonLanguage.getLogger().log(Level.FINE, "write of Python struct native member " + key);
+        }
+
         protected boolean eq(String expected, String actual) {
             return expected.equals(actual);
         }

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/PRootNode.java

@@ -40,7 +40,7 @@
  */
 package com.oracle.graal.python.nodes;
 
-import com.oracle.truffle.api.CompilerDirectives;
+import com.oracle.truffle.api.CompilerAsserts;
 import com.oracle.truffle.api.CompilerDirectives.CompilationFinal;
 import com.oracle.truffle.api.TruffleLanguage;
 import com.oracle.truffle.api.frame.FrameDescriptor;
@@ -62,8 +62,8 @@ public boolean needsCallerFrame() {
     }
 
     public void setNeedsCallerFrame() {
+        CompilerAsserts.neverPartOfCompilation("this is usually called from behind a TruffleBoundary");
         if (!this.needsCallerFrame) {
-            CompilerDirectives.transferToInterpreterAndInvalidate();
             this.needsCallerFrame = true;
         }
     }