[GR-13467] GetItemNode should raise TypeError not return NotImplemented value

Author: ppisl

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/subscript/GetItemNode.java

@@ -32,14 +32,17 @@
 import com.oracle.graal.python.nodes.expression.ExpressionNode;
 import com.oracle.graal.python.nodes.frame.ReadNode;
 import com.oracle.graal.python.nodes.statement.StatementNode;
-import com.oracle.truffle.api.dsl.Cached;
+import static com.oracle.graal.python.runtime.exception.PythonErrorType.TypeError;
+import com.oracle.truffle.api.CompilerDirectives;
 import com.oracle.truffle.api.dsl.Specialization;
 import com.oracle.truffle.api.frame.VirtualFrame;
 import com.oracle.truffle.api.nodes.NodeInfo;
 
 @NodeInfo(shortName = __GETITEM__)
 public abstract class GetItemNode extends BinaryOpNode implements ReadNode {
 
+    @Child private LookupAndCallBinaryNode callGetitemNode;
+
     public ExpressionNode getPrimary() {
         return getLeftNode();
     }
@@ -53,8 +56,16 @@ public ExpressionNode getSlice() {
     public abstract Object execute(Object primary, Object slice);
 
     @Specialization
-    public Object doSpecialObject(Object primary, Object index,
-                    @Cached("create(__GETITEM__)") LookupAndCallBinaryNode callGetitemNode) {
+    public Object doSpecialObject(Object primary, Object index) {
+        if (callGetitemNode == null) {
+            CompilerDirectives.transferToInterpreterAndInvalidate();
+            callGetitemNode = insert(LookupAndCallBinaryNode.create(__GETITEM__, null, () -> new LookupAndCallBinaryNode.NotImplementedHandler() {
+                @Override
+                public Object execute(@SuppressWarnings("unused") Object arg, Object arg2) {
+                    throw raise(TypeError, "'%p' object is not subscriptable", arg2);
+                }
+            }));
+        }
         return callGetitemNode.executeObject(primary, index);
     }