adapt GetAttributeNode builtin to the new builtin LazyPythonClass mechanism

Author: cosminbasca

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/object/ObjectBuiltins.java

@@ -81,6 +81,7 @@
 import com.oracle.graal.python.nodes.function.builtins.PythonVarargsBuiltinNode;
 import com.oracle.graal.python.nodes.object.GetClassNode;
 import com.oracle.graal.python.nodes.object.GetLazyClassNode;
+import com.oracle.graal.python.nodes.object.IsBuiltinClassProfile;
 import com.oracle.truffle.api.CompilerDirectives;
 import com.oracle.truffle.api.CompilerDirectives.TruffleBoundary;
 import com.oracle.truffle.api.dsl.Cached;
@@ -478,8 +479,10 @@ protected PNone doIt(Object object, Object key,
     @Builtin(name = __DICT__, minNumOfPositionalArgs = 1, maxNumOfPositionalArgs = 2, isGetter = true, isSetter = true)
     @GenerateNodeFactory
     static abstract class DictNode extends PythonBinaryBuiltinNode {
+        private final IsBuiltinClassProfile isBuiltinClassProfile = IsBuiltinClassProfile.create();
+
         protected boolean isExactObjectInstance(PythonObject self) {
-            return lookupClass(PythonBuiltinClassType.PythonObject) == self.getPythonClass();
+            return isBuiltinClassProfile.profileObject(self, PythonBuiltinClassType.PythonObject);
         }
 
         @SuppressWarnings("unused")

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/attributes/GetAttributeNode.java

@@ -47,6 +47,7 @@
 import com.oracle.graal.python.nodes.call.special.LookupAndCallBinaryNode;
 import com.oracle.graal.python.nodes.expression.ExpressionNode;
 import com.oracle.graal.python.nodes.frame.ReadNode;
+import com.oracle.graal.python.nodes.object.IsBuiltinClassProfile;
 import com.oracle.graal.python.nodes.statement.StatementNode;
 import com.oracle.graal.python.runtime.exception.PException;
 import com.oracle.truffle.api.CompilerDirectives;
@@ -63,7 +64,7 @@ public abstract class GetAttributeNode extends ExpressionNode implements ReadNod
 
     @Child LookupAndCallBinaryNode dispatchNode = LookupAndCallBinaryNode.create(__GETATTRIBUTE__);
     @Child LookupAndCallBinaryNode dispatchGetAttr;
-    @CompilationFinal private ConditionProfile errorProfile = ConditionProfile.createBinaryProfile();
+    @CompilationFinal private IsBuiltinClassProfile isBuiltinClassProfile = IsBuiltinClassProfile.create();
 
     protected GetAttributeNode(String key) {
         this.key = key;
@@ -88,7 +89,7 @@ protected int doItInt(Object object) throws UnexpectedResultException {
         try {
             return dispatchNode.executeInt(object, key);
         } catch (PException pe) {
-            pe.expect(AttributeError, getCore(), errorProfile);
+            pe.expect(AttributeError, isBuiltinClassProfile);
             return getDispatchGetAttr().executeInt(object, key);
         }
     }
@@ -98,7 +99,7 @@ protected boolean doItBoolean(Object object) throws UnexpectedResultException {
         try {
             return dispatchNode.executeBool(object, key);
         } catch (PException pe) {
-            pe.expect(AttributeError, getCore(), errorProfile);
+            pe.expect(AttributeError, isBuiltinClassProfile);
             return getDispatchGetAttr().executeBool(object, key);
         }
     }
@@ -108,7 +109,7 @@ protected Object doIt(Object object) {
         try {
             return dispatchNode.executeObject(object, key);
         } catch (PException pe) {
-            pe.expect(AttributeError, getCore(), errorProfile);
+            pe.expect(AttributeError, isBuiltinClassProfile);
             return getDispatchGetAttr().executeObject(object, key);
         }
     }