No more specializeForInterpretation. All specialization is part of IRBuilding

Author: enebo

core/src/main/java/org/jruby/ir/IRBuilder.java

@@ -839,7 +839,7 @@ private Operand buildAttrAssign(final AttrAssignNode attrAssignNode, IRScope s)
         List<Operand> args = new ArrayList<>();
         Node argsNode = attrAssignNode.getArgsNode();
         Operand lastArg = (argsNode == null) ? manager.getNil() : buildAttrAssignCallArgs(args, argsNode, s);
-        addInstr(s, new AttrAssignInstr(obj, attrAssignNode.getName(), args.toArray(new Operand[args.size()])));
+        addInstr(s, AttrAssignInstr.create(obj, attrAssignNode.getName(), args.toArray(new Operand[args.size()])));
         return lastArg;
     }
 
@@ -848,7 +848,7 @@ public Operand buildAttrAssignAssignment(Node node, IRScope s, Operand value) {
         Operand obj = build(attrAssignNode.getReceiverNode(), s);
         Operand[] args = setupCallArgs(attrAssignNode.getArgsNode(), s);
         args = addArg(args, value);
-        addInstr(s, new AttrAssignInstr(obj, attrAssignNode.getName(), args));
+        addInstr(s, AttrAssignInstr.create(obj, attrAssignNode.getName(), args));
         return value;
     }
 

core/src/main/java/org/jruby/ir/IRScope.java

@@ -486,11 +486,6 @@ protected Instr[] prepareInstructions() {
                     // FIXME: Can be removed once ipc and rpc are stored in table(s) in IC
                     Instr newInstr = instr.clone(cloneInfo);
 
-                    // FIXME: Can be removed once noresult and attrassign properly specialize at IRBuild time.
-                    if (newInstr instanceof Specializeable) {
-                        newInstr = ((Specializeable) newInstr).specializeForInterpretation();
-                    }
-
                     newInstr.setIPC(ipc);
                     newInstrs.add(newInstr);
                     ipc++;

core/src/main/java/org/jruby/ir/instructions/AttrAssignInstr.java

@@ -15,12 +15,16 @@
 // Instruction representing Ruby code of the form: "a[i] = 5"
 // which is equivalent to: a.[](i,5)
 public class AttrAssignInstr extends NoResultCallInstr {
-    public AttrAssignInstr(Operand obj, String attr, Operand[] args) {
-        super(Operation.ATTR_ASSIGN, CallType.UNKNOWN, attr, obj, args, null);
+    public static AttrAssignInstr create(Operand obj, String attr, Operand[] args) {
+        if (!containsArgSplat(args) && args.length == 1) {
+            return new OneArgOperandAttrAssignInstr(obj, attr, args);
+        }
+
+        return new AttrAssignInstr(obj, attr, args);
     }
 
-    public AttrAssignInstr(AttrAssignInstr instr) {
-        this(instr.getReceiver(), instr.getName(), instr.getCallArgs());
+    public AttrAssignInstr(Operand obj, String attr, Operand[] args) {
+        super(Operation.ATTR_ASSIGN, CallType.UNKNOWN, attr, obj, args, null);
     }
 
     @Override
@@ -39,18 +43,6 @@ public Instr clone(CloneInfo ii) {
         return new AttrAssignInstr(receiver.cloneForInlining(ii), getName(), cloneCallArgs(ii));
     }
 
-    @Override
-    public CallBase specializeForInterpretation() {
-        Operand[] callArgs = getCallArgs();
-        if (containsArgSplat(callArgs)) return this;
-
-        switch (callArgs.length) {
-            case 1:
-                return new OneArgOperandAttrAssignInstr(this);
-        }
-        return this;
-    }
-
     @Override
     public Object interpret(ThreadContext context, StaticScope currScope, DynamicScope dynamicScope, IRubyObject self, Object[] temp) {
         IRubyObject object = (IRubyObject) receiver.retrieve(context, self, currScope, dynamicScope, temp);

core/src/main/java/org/jruby/ir/instructions/CallBase.java

@@ -18,7 +18,7 @@
 
 import static org.jruby.ir.IRFlags.*;
 
-public abstract class CallBase extends Instr implements Specializeable, ClosureAcceptingInstr {
+public abstract class CallBase extends Instr implements ClosureAcceptingInstr {
     private static long callSiteCounter = 1;
 
     public final long callSiteId;
@@ -211,15 +211,6 @@ public boolean computeScopeFlags(IRScope scope) {
 
         return modifiedScope;
     }
-    /**
-     * Interpreter can ask the instruction if it knows how to make a more
-     * efficient instruction for direct interpretation.
-     *
-     * @return itself or more efficient but semantically equivalent instr
-     */
-    public CallBase specializeForInterpretation() {
-        return this;
-    }
 
     @Override
     public void simplifyOperands(Map<Operand, Operand> valueMap, boolean force) {

core/src/main/java/org/jruby/ir/instructions/CallInstr.java

@@ -68,7 +68,7 @@ public void updateResult(Variable v) {
     }
 
     public Instr discardResult() {
-        return new NoResultCallInstr(Operation.NORESULT_CALL, getCallType(), getName(), getReceiver(), getCallArgs(), closure);
+        return NoResultCallInstr.create(getCallType(), getName(), getReceiver(), getCallArgs(), closure);
     }
 
     @Override

core/src/main/java/org/jruby/ir/instructions/ClassSuperInstr.java

@@ -32,11 +32,6 @@ public Instr discardResult() {
         return this;
     }
 
-    @Override
-    public CallBase specializeForInterpretation() {
-        return this;
-    }
-
     @Override
     public Object interpret(ThreadContext context, StaticScope currScope, DynamicScope currDynScope, IRubyObject self, Object[] temp) {
         IRubyObject[] args = prepareArguments(context, self, getCallArgs(), currScope, currDynScope, temp);

core/src/main/java/org/jruby/ir/instructions/ConstMissingInstr.java

@@ -45,12 +45,6 @@ public void updateResult(Variable v) {
         this.result = v;
     }
 
-    // we don't want to convert const_missing to an actual call
-    @Override
-    public CallBase specializeForInterpretation() {
-        return this;
-    }
-
     @Override
     public Instr clone(CloneInfo ii) {
         return new ConstMissingInstr(ii.getRenamedVariable(result), receiver.cloneForInlining(ii), missingConst);

core/src/main/java/org/jruby/ir/instructions/InstanceSuperInstr.java

@@ -35,11 +35,6 @@ public Instr discardResult() {
         return this;
     }
 
-    @Override
-    public CallBase specializeForInterpretation() {
-        return this;
-    }
-
     @Override
     public Object interpret(ThreadContext context, StaticScope currScope, DynamicScope currDynScope, IRubyObject self, Object[] temp) {
         IRubyObject[] args = prepareArguments(context, self, getCallArgs(), currScope, currDynScope, temp);

core/src/main/java/org/jruby/ir/instructions/NoResultCallInstr.java

@@ -8,12 +8,17 @@
 import org.jruby.runtime.CallType;
 
 public class NoResultCallInstr extends CallBase {
-    public NoResultCallInstr(Operation op, CallType callType, String name, Operand receiver, Operand[] args, Operand closure) {
-        super(op, callType, name, receiver, args, closure);
+    // FIXME: Removed results undoes specialized callinstrs.  Audit how often and what and make equalivalent versions here.
+    public static NoResultCallInstr create(CallType callType, String name, Operand receiver, Operand[] args, Operand closure) {
+        if (closure == null && !containsArgSplat(args) && args.length == 1) {
+            return new OneOperandArgNoBlockNoResultCallInstr(callType, name, receiver, args, closure);
+        }
+
+        return new NoResultCallInstr(Operation.NORESULT_CALL, callType, name, receiver, args, closure);
     }
 
-    public NoResultCallInstr(Operation op, NoResultCallInstr instr) {
-        this(op, instr.getCallType(), instr.getName(), instr.receiver, instr.arguments, instr.closure);
+    public NoResultCallInstr(Operation op, CallType callType, String name, Operand receiver, Operand[] args, Operand closure) {
+        super(op, callType, name, receiver, args, closure);
     }
 
     @Override
@@ -22,22 +27,6 @@ public Instr clone(CloneInfo ii) {
                 cloneCallArgs(ii), closure == null ? null : closure.cloneForInlining(ii));
     }
 
-    @Override
-    public CallBase specializeForInterpretation() {
-        Operand[] callArgs = getCallArgs();
-        if (hasClosure() || containsArgSplat(callArgs)) return this;
-
-        switch (callArgs.length) {
-//            case 0:
-//                return new ZeroOperandArgNoBlockNoResultCallInstr(this);
-            case 1:
-//                if (isAllFixnums()) return new OneFixnumArgNoBlockNoResultCallInstr(this);
-
-                return new OneOperandArgNoBlockNoResultCallInstr(this);
-        }
-        return this;
-    }
-
     @Override
     public void visit(IRVisitor visitor) {
         visitor.NoResultCallInstr(this);