Address review comments

Author: msimacek

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

@@ -597,7 +597,7 @@ private WriteAttributeToObjectNode ensureWriteNode() {
     @Builtin(name = __DELATTR__, minNumOfPositionalArgs = 2)
     @GenerateNodeFactory
     public abstract static class DelattrNode extends PythonBinaryBuiltinNode {
-        @Child GetClassNode getDescClassNode;
+        @Child private GetClassNode getDescClassNode;
 
         @Specialization
         protected PNone doIt(VirtualFrame frame, Object object, Object keyObj,

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

@@ -129,11 +129,6 @@ public void deleteDict(Object receiver) throws UnsupportedMessageException {
         throw UnsupportedMessageException.create();
     }
 
-    // Temporary class getter before equals gets converted to a node
-    private Object getPythonClass(Object receiver) {
-        return getDefaultNodes().getGetClassNode().execute(receiver);
-    }
-
     /**
      * @return true if the receiver of a Python type.<br>
      *         <br>
@@ -461,8 +456,8 @@ public boolean equalsWithState(Object receiver, Object other, PythonObjectLibrar
 
         boolean checkedReverseOp = false;
 
-        Object leftClass = getPythonClass(receiver);
-        Object rightClass = otherLibrary.getPythonClass(other);
+        Object leftClass = getDefaultNodes().getGetClassNode().execute(receiver);
+        Object rightClass = otherLibrary.getDefaultNodes().getGetClassNode().execute(other);
         int result;
         boolean isSameType = getDefaultNodes().getIsSameTypeNode().execute(leftClass, rightClass);
         if (!isSameType && getDefaultNodes().getIsSubtypeNode().execute(rightClass, leftClass)) {
@@ -1071,8 +1066,9 @@ public Object getDelegatedValue(Object receiver) {
      * @param type the class to be checked against
      */
     public boolean typeCheck(Object receiver, Object type) {
-        return getDefaultNodes().getIsSameTypeNode().execute(getPythonClass(receiver), type) ||
-                        getDefaultNodes().getIsSubtypeNode().execute(getPythonClass(receiver), type);
+        Object clazz = getDefaultNodes().getGetClassNode().execute(receiver);
+        return getDefaultNodes().getIsSameTypeNode().execute(clazz, type) ||
+                        getDefaultNodes().getIsSubtypeNode().execute(clazz, type);
     }
 
     /**

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/thread/LockBuiltins.java

@@ -252,22 +252,20 @@ boolean isLocked(PLock self) {
     @GenerateNodeFactory
     abstract static class ReprLockNode extends PythonUnaryBuiltinNode {
         @Specialization
-        String repr(PLock self,
-                        @Cached GetClassNode getClassNode,
-                        @Cached GetNameNode getNameNode) {
+        @TruffleBoundary
+        String repr(PLock self) {
             return PythonUtils.format("<%s %s object at %s>",
                             (self.locked()) ? "locked" : "unlocked",
-                            getNameNode.execute(getClassNode.execute(self)),
+                            GetNameNode.getUncached().execute(GetClassNode.getUncached().execute(self)),
                             self.hashCode());
         }
 
         @Specialization
-        String repr(PRLock self,
-                        @Cached GetClassNode getClassNode,
-                        @Cached GetNameNode getNameNode) {
+        @TruffleBoundary
+        String repr(PRLock self) {
             return PythonUtils.format("<%s %s object owner=%d count=%d at %s>",
                             (self.locked()) ? "locked" : "unlocked",
-                            getNameNode.execute(getClassNode.execute(self)),
+                            GetNameNode.getUncached().execute(GetClassNode.getUncached().execute(self)),
                             self.getOwnerId(),
                             self.getCount(),
                             self.hashCode());