[GR-21265] In EmulateJython mode isinstance function should behave in different way.

PullRequest: graalpython/822

Author: ppisl

graalpython/com.oracle.graal.python.test/src/tests/test_interop.py

@@ -490,3 +490,17 @@ def test_java_null_is_none():
             assert (x is z) == False
             assert x is not z
 
+    def test_isinstance01():
+        if sys.graal_python_jython_emulation_enabled:
+            import java.lang.Integer as Integer
+            i = Integer(1)
+            assert isinstance(i, Integer)
+
+    def test_isinstance02():
+        if sys.graal_python_jython_emulation_enabled:
+            import java.util.Map as Map
+            import java.util.HashMap as HashMap
+            h = HashMap()
+            assert isinstance(h, HashMap)
+            assert isinstance(h, Map)
+        

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

@@ -180,6 +180,7 @@
 import com.oracle.truffle.api.CompilerDirectives.TruffleBoundary;
 import com.oracle.truffle.api.RootCallTarget;
 import com.oracle.truffle.api.Truffle;
+import com.oracle.truffle.api.TruffleLanguage;
 import com.oracle.truffle.api.debug.Debugger;
 import com.oracle.truffle.api.dsl.Cached;
 import com.oracle.truffle.api.dsl.Cached.Shared;
@@ -1056,6 +1057,8 @@ public abstract static class IsInstanceNode extends PythonBinaryBuiltinNode {
         @Child private SequenceStorageNodes.LenNode lenNode;
         @Child private GetObjectArrayNode getObjectArrayNode;
 
+        @CompilationFinal private Boolean emulateJython;
+
         public static IsInstanceNode create() {
             return BuiltinFunctionsFactory.IsInstanceNodeFactory.create();
         }
@@ -1101,8 +1104,22 @@ boolean isInstance(VirtualFrame frame, Object instance, PTuple clsTuple,
             return false;
         }
 
+        protected boolean emulateJython() {
+            if (emulateJython == null) {
+                CompilerDirectives.transferToInterpreterAndInvalidate();
+                emulateJython = PythonOptions.getFlag(getContext(), PythonOptions.EmulateJython);
+            }
+            return emulateJython;
+        }
+
         @Fallback
         boolean isInstance(VirtualFrame frame, Object instance, Object cls) {
+            TruffleLanguage.Env env = getContext().getEnv();
+            if (emulateJython() && env.isHostObject(cls)) {
+                Object hostCls = env.asHostObject(cls);
+                Object hostInstance = env.isHostObject(instance) ? env.asHostObject(instance) : instance;
+                return hostCls instanceof Class && ((Class<?>) hostCls).isAssignableFrom(hostInstance.getClass());
+            }
             return isInstanceCheckInternal(frame, instance, cls) || typeInstanceCheckNode.executeWith(frame, cls, instance);
         }