getting the node factory is now on the fast path

Author: timfel

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/function/PBuiltinFunction.java

@@ -82,7 +82,6 @@ public RootNode getFunctionRootNode() {
     }
 
     public NodeFactory<? extends PythonBuiltinBaseNode> getBuiltinNodeFactory() {
-        CompilerAsserts.neverPartOfCompilation();
         RootNode functionRootNode = getFunctionRootNode();
         if (functionRootNode instanceof BuiltinFunctionRootNode) {
             return ((BuiltinFunctionRootNode) functionRootNode).getFactory();

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

@@ -61,10 +61,12 @@
 import com.oracle.truffle.api.TruffleLanguage.ContextReference;
 import com.oracle.truffle.api.dsl.Cached;
 import com.oracle.truffle.api.dsl.ImportStatic;
+import com.oracle.truffle.api.dsl.ReportPolymorphism;
 import com.oracle.truffle.api.dsl.Specialization;
 import com.oracle.truffle.api.nodes.ExplodeLoop;
 
 @ImportStatic(PythonOptions.class)
+@ReportPolymorphism
 public abstract class LookupAttributeInMRONode extends PNodeWithContext {
     public abstract static class Dynamic extends PNodeWithContext {
         private static final DynamicUncached UNCACHED = new DynamicUncached();

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/function/builtins/WrapTpNew.java

@@ -44,7 +44,6 @@
 import com.oracle.graal.python.builtins.PythonBuiltinClassType;
 import com.oracle.graal.python.builtins.objects.function.PArguments;
 import com.oracle.graal.python.builtins.objects.function.PBuiltinFunction;
-import com.oracle.graal.python.builtins.objects.type.TypeNodes.GetBaseClassesNode;
 import com.oracle.graal.python.builtins.objects.type.TypeNodes.IsTypeNode;
 import com.oracle.graal.python.nodes.PRaiseNode;
 import com.oracle.graal.python.nodes.SpecialMethodNames;
@@ -56,16 +55,17 @@
 import com.oracle.truffle.api.dsl.NodeFactory;
 import com.oracle.truffle.api.frame.VirtualFrame;
 import com.oracle.truffle.api.nodes.NodeCost;
+import com.oracle.truffle.api.profiles.ValueProfile;
 
 /**
  * Implements cpython://Objects/typeobject.c#tp_new_wrapper.
  */
 public final class WrapTpNew extends SlotWrapper {
     @Child IsTypeNode isType;
     @Child IsSubtypeNode isSubtype;
-    @Child GetBaseClassesNode getBases;
     @Child PRaiseNode raiseNode;
     @Child LookupAttributeInMRONode lookupNewNode;
+    @CompilationFinal ValueProfile builtinProfile;
     @CompilationFinal byte state = 0;
     @CompilationFinal PythonBuiltinClassType owner;
 
@@ -122,12 +122,17 @@ public Object execute(VirtualFrame frame) {
             // we're in. We have our optimizations for this lookup that the compiler can then
             // (hopefully) merge with the initial lookup of the new method before entering it.
             if (lookupNewNode == null) {
+                reportPolymorphicSpecialize();
                 CompilerDirectives.transferToInterpreterAndInvalidate();
                 lookupNewNode = insert(LookupAttributeInMRONode.createForLookupOfUnmanagedClasses(SpecialMethodNames.__NEW__));
             }
             Object newMethod = lookupNewNode.execute(arg0);
             if (newMethod instanceof PBuiltinFunction) {
-                NodeFactory<? extends PythonBuiltinBaseNode> factory = ((PBuiltinFunction) newMethod).getBuiltinNodeFactory();
+                if (builtinProfile == null) {
+                    CompilerDirectives.transferToInterpreterAndInvalidate();
+                    builtinProfile = ValueProfile.createIdentityProfile();
+                }
+                NodeFactory<? extends PythonBuiltinBaseNode> factory = ((PBuiltinFunction) builtinProfile.profile(newMethod)).getBuiltinNodeFactory();
                 if (factory != null) {
                     if (!factory.getNodeClass().isAssignableFrom(getNode().getClass())) {
                         if ((state & IS_UNSAFE_STATE) == 0) {