Use CallBinaryMethodNode for dispatching __getattribute__ callable

It is a common pattern that Python class defines only __getattr__, in which
case the TpSlotGetAttrPython will wrap the inherited builtin __getattribute__
and the user defined __getattr__ callable.

Normally, slots wrapping Python callables do not wrap builtins. This is
impossible for slots wrapping one callable, because we unwrap them in
fixupSlotDispatchers (like CPython). For other slots wrapping 2 callables it is
probably not as common as for tp_getattro. Therefore the dispatching code for
those nodes, does not have a fast-path for builtin callables. This PR adds such
fast-path to CallManagedSlotGetAttrNode.

Author: steve-s

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/type/slots/TpSlotGetAttr.java

@@ -63,6 +63,8 @@
 import com.oracle.graal.python.builtins.objects.type.slots.TpSlot.TpSlotPython;
 import com.oracle.graal.python.nodes.PGuards;
 import com.oracle.graal.python.nodes.attributes.LookupAttributeInMRONode.Dynamic;
+import com.oracle.graal.python.nodes.call.special.CallBinaryMethodNode;
+import com.oracle.graal.python.nodes.call.special.MaybeBindDescriptorNode;
 import com.oracle.graal.python.nodes.function.builtins.PythonBinaryBuiltinNode;
 import com.oracle.graal.python.nodes.object.BuiltinClassProfiles.IsBuiltinObjectProfile;
 import com.oracle.graal.python.runtime.ExecutionContext.CallContext;
@@ -297,6 +299,8 @@ static Object callPythonSimple(VirtualFrame frame, Node inliningTarget, TpSlotGe
 
         @Specialization(guards = "slot.hasGetattr()")
         static Object callPythonSimple(VirtualFrame frame, Node inliningTarget, TpSlotGetAttrPython slot, Object self, Object name,
+                        @Cached MaybeBindDescriptorNode bindDescriptorNode,
+                        @Cached CallBinaryMethodNode callGetAttributeNode,
                         @Exclusive @Cached BinaryPythonSlotDispatcherNode callPythonFun,
                         @Cached IsBuiltinObjectProfile errorProfile) {
             // equivalent of slot_tp_getattr_hook
@@ -309,7 +313,13 @@ static Object callPythonSimple(VirtualFrame frame, Node inliningTarget, TpSlotGe
                 // TODO: CPython calls PyObject_GenericGetAttr if there is no __getattribute__. Can
                 // we create a type that does not inherit tp_getattro and so no __getattribute__ is
                 // created for it in add_operators?
-                return callPythonFun.execute(frame, inliningTarget, slot.getGetattribute(), type, self, name);
+
+                // NOTE: we use CallBinaryMethodNode, because it is common that the __getattribute__
+                // callable is PBuiltinFunction wrapper is wrapping raw tp_getattro slot. This
+                // happens if a Python class declares __getattr__ and inherits __getattribute__
+                // wrapping some builtin slot.
+                Object bound = bindDescriptorNode.execute(frame, inliningTarget, slot.getGetattribute(), self, type);
+                return callGetAttributeNode.executeObject(frame, bound, self, name);
             } catch (PException pe) {
                 pe.expect(inliningTarget, AttributeError, errorProfile);
                 if (getattr == null) {