Simplify 'IS_POINTER' MR.

Author: fangerer

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

@@ -63,7 +63,6 @@
 import com.oracle.graal.python.builtins.objects.cext.NativeWrappers.PythonNativeWrapper;
 import com.oracle.graal.python.builtins.objects.cext.NativeWrappers.PythonObjectNativeWrapper;
 import com.oracle.graal.python.builtins.objects.cext.PythonObjectNativeWrapperMRFactory.PAsPointerNodeGen;
-import com.oracle.graal.python.builtins.objects.cext.PythonObjectNativeWrapperMRFactory.PIsPointerNodeGen;
 import com.oracle.graal.python.builtins.objects.cext.PythonObjectNativeWrapperMRFactory.ReadNativeMemberNodeGen;
 import com.oracle.graal.python.builtins.objects.cext.PythonObjectNativeWrapperMRFactory.ToPyObjectNodeGen;
 import com.oracle.graal.python.builtins.objects.cext.PythonObjectNativeWrapperMRFactory.WriteNativeMemberNodeGen;
@@ -850,18 +849,17 @@ public Object access(Object object) {
     @Resolve(message = "TO_NATIVE")
     abstract static class ToNativeNode extends Node {
         @Child private ToPyObjectNode toPyObjectNode;
-        @Child private PIsPointerNode pIsPointerNode = PIsPointerNode.create();
 
         Object access(PythonClassInitNativeWrapper obj) {
-            if (!pIsPointerNode.execute(obj)) {
+            if (!obj.isNative()) {
                 obj.setNativePointer(getToPyObjectNode().execute(obj));
             }
             return obj;
         }
 
         Object access(PythonNativeWrapper obj) {
             assert !(obj instanceof PythonClassInitNativeWrapper);
-            if (!pIsPointerNode.execute(obj)) {
+            if (!obj.isNative()) {
                 obj.setNativePointer(getToPyObjectNode().execute(obj));
             }
             return obj;
@@ -878,10 +876,8 @@ private ToPyObjectNode getToPyObjectNode() {
 
     @Resolve(message = "IS_POINTER")
     abstract static class IsPointerNode extends Node {
-        @Child private PIsPointerNode pIsPointerNode = PIsPointerNode.create();
-
         boolean access(PythonNativeWrapper obj) {
-            return pIsPointerNode.execute(obj);
+            return obj.isNative();
         }
     }
 
@@ -894,29 +890,6 @@ long access(PythonNativeWrapper obj) {
         }
     }
 
-    abstract static class PIsPointerNode extends PNodeWithContext {
-
-        public abstract boolean execute(PythonNativeWrapper obj);
-
-        @Specialization(guards = "!obj.isNative()")
-        boolean doBool(BoolNativeWrapper obj) {
-            // Special case: Booleans are singletons, so we need to check if the singletons have
-            // native wrappers associated and if they are already native.
-            PInt boxed = factory().createInt(obj.getValue());
-            DynamicObjectNativeWrapper nativeWrapper = boxed.getNativeWrapper();
-            return nativeWrapper != null && nativeWrapper.isNative();
-        }
-
-        @Fallback
-        boolean doBool(PythonNativeWrapper obj) {
-            return obj.isNative();
-        }
-
-        private static PIsPointerNode create() {
-            return PIsPointerNodeGen.create();
-        }
-    }
-
     abstract static class PAsPointerNode extends PNodeWithContext {
         @Child private Node asPointerNode;