[GR-25584] Introduce message PythonObjectLibrary.getIterator

PullRequest: graalpython/1250

Author: fangerer

graalpython/com.oracle.graal.python.test/src/com/oracle/graal/python/test/datatype/PRangeTests.java

@@ -28,13 +28,13 @@
 import static com.oracle.graal.python.test.PythonTests.assertPrints;
 import static org.junit.Assert.assertEquals;
 
-import com.oracle.graal.python.builtins.objects.range.PIntRange;
 import org.junit.Before;
 import org.junit.Test;
 
 import com.oracle.graal.python.PythonLanguage;
+import com.oracle.graal.python.builtins.objects.object.PythonObjectLibrary;
+import com.oracle.graal.python.builtins.objects.range.PIntRange;
 import com.oracle.graal.python.builtins.objects.range.PRange;
-import com.oracle.graal.python.nodes.control.GetIteratorExpressionNode.GetIteratorNode;
 import com.oracle.graal.python.nodes.control.GetNextNode;
 import com.oracle.graal.python.nodes.object.IsBuiltinClassProfile;
 import com.oracle.graal.python.runtime.exception.PException;
@@ -71,9 +71,7 @@ public void loopWithOnlyStop() throws UnexpectedResultException {
         PRange range = PythonObjectFactory.getUncached().createIntRange(10);
         int index = 0;
         TestRoot testRoot = new TestRoot(PythonLanguage.getCurrent());
-        GetIteratorNode getIter = GetIteratorNode.create();
-        testRoot.doInsert(getIter);
-        Object iter = getIter.executeWith(null, range);
+        Object iter = PythonObjectLibrary.getUncached().getIterator(range);
         GetNextNode next = GetNextNode.create();
         testRoot.doInsert(next);
         IsBuiltinClassProfile errorProfile = IsBuiltinClassProfile.getUncached();
@@ -95,9 +93,7 @@ public void loopWithStep() throws UnexpectedResultException {
         PRange range = PythonObjectFactory.getUncached().createIntRange(0, 10, 2, 5);
         int index = 0;
         TestRoot testRoot = new TestRoot(PythonLanguage.getCurrent());
-        GetIteratorNode getIter = GetIteratorNode.create();
-        testRoot.doInsert(getIter);
-        Object iter = getIter.executeWith(null, range);
+        Object iter = PythonObjectLibrary.getUncached().getIterator(range);
         GetNextNode next = GetNextNode.create();
         testRoot.doInsert(next);
         IsBuiltinClassProfile errorProfile = IsBuiltinClassProfile.getUncached();

graalpython/com.oracle.graal.python.test/testData/goldenFiles/AwaitAndAsyncTests/asyncFor01.tast

@@ -45,7 +45,6 @@ ModuleRootNode Name: <module 'asyncFor01'> SourceSection: [0,39]`async def f():
                             ExpressionStatementNode SourceSection: [35,39]`pass`
                               EmptyNode SourceSection: [35,39]`pass`
                           GetIteratorExpressionNodeGen SourceSection: [31,33]`()`
-                            GetIteratorNodeGen SourceSection: None
                             TupleLiteralNode SourceSection: [31,33]`()`
                               PythonObjectFactoryNodeGen SourceSection: None
                         BlockNode SourceSection: None

graalpython/com.oracle.graal.python.test/testData/goldenFiles/AwaitAndAsyncTests/asyncFor02.tast

@@ -64,7 +64,6 @@ ModuleRootNode Name: <module 'asyncFor02'> SourceSection: [0,42]`async def f():
                             ExpressionStatementNode SourceSection: [38,42]`pass`
                               EmptyNode SourceSection: [38,42]`pass`
                           GetIteratorExpressionNodeGen SourceSection: [34,36]`()`
-                            GetIteratorNodeGen SourceSection: None
                             TupleLiteralNode SourceSection: [34,36]`()`
                               PythonObjectFactoryNodeGen SourceSection: None
                         BlockNode SourceSection: None

graalpython/com.oracle.graal.python.test/testData/goldenFiles/BasicTests/for01.tast

@@ -18,6 +18,5 @@ ModuleRootNode Name: <module 'for01'> SourceSection: [0,23]`for i in 'ahoj':↵
               ExpressionStatementNode SourceSection: [19,23]`pass`
                 EmptyNode SourceSection: [19,23]`pass`
             GetIteratorExpressionNodeGen SourceSection: [9,15]`'ahoj'`
-              GetIteratorNodeGen SourceSection: None
               StringLiteralNode SourceSection: [9,15]`'ahoj'`
           BlockNode SourceSection: None

graalpython/com.oracle.graal.python.test/testData/goldenFiles/BasicTests/for02.tast

@@ -27,7 +27,6 @@ ModuleRootNode Name: <module 'for02'> SourceSection: [0,31]`for i in range(210):
                     IsBuiltinClassProfile SourceSection: None
                       CachedDispatchFirst SourceSection: None
             GetIteratorExpressionNodeGen SourceSection: [9,19]`range(210)`
-              GetIteratorNodeGen SourceSection: None
               PythonCallUnary SourceSection: [9,19]`range(210)`
                 CallUnaryMethodNodeGen SourceSection: None
                 ReadNameNodeGen SourceSection: [9,14]`range`

graalpython/com.oracle.graal.python.test/testData/goldenFiles/BasicTests/for03.tast

@@ -31,7 +31,6 @@ ModuleRootNode Name: <module 'for03'> SourceSection: [0,43]`for x in xrange(3):
                 BreakNode SourceSection: [37,42]`break`
                 BlockNode SourceSection: None
             GetIteratorExpressionNodeGen SourceSection: [9,18]`xrange(3)`
-              GetIteratorNodeGen SourceSection: None
               PythonCallUnary SourceSection: [9,18]`xrange(3)`
                 CallUnaryMethodNodeGen SourceSection: None
                 ReadNameNodeGen SourceSection: [9,15]`xrange`

graalpython/com.oracle.graal.python.test/testData/goldenFiles/BasicTests/for04.tast

@@ -57,7 +57,6 @@ ModuleRootNode Name: <module 'for04'> SourceSection: [0,59]`def fn():↵  for x
                               BreakNode SourceSection: [53,58]`break`
                               BlockNode SourceSection: None
                           GetIteratorExpressionNodeGen SourceSection: [21,30]`xrange(3)`
-                            GetIteratorNodeGen SourceSection: None
                             PythonCallUnary SourceSection: [21,30]`xrange(3)`
                               CallUnaryMethodNodeGen SourceSection: None
                               ReadGlobalOrBuiltinNodeGen SourceSection: [21,27]`xrange`

graalpython/com.oracle.graal.python.test/testData/goldenFiles/BasicTests/for07.tast

@@ -49,7 +49,6 @@ ModuleRootNode Name: <module 'for07'> SourceSection: [0,68]`for x in range(10):
                         IsBuiltinClassProfile SourceSection: None
                           CachedDispatchFirst SourceSection: None
             GetIteratorExpressionNodeGen SourceSection: [9,18]`range(10)`
-              GetIteratorNodeGen SourceSection: None
               PythonCallUnary SourceSection: [9,18]`range(10)`
                 CallUnaryMethodNodeGen SourceSection: None
                 ReadNameNodeGen SourceSection: [9,14]`range`

graalpython/com.oracle.graal.python.test/testData/goldenFiles/BasicTests/for08.tast

@@ -46,7 +46,6 @@ ModuleRootNode Name: <module 'for08'> SourceSection: [0,71]`for x in range(10):
                         IsBuiltinClassProfile SourceSection: None
                           CachedDispatchFirst SourceSection: None
             GetIteratorExpressionNodeGen SourceSection: [9,18]`range(10)`
-              GetIteratorNodeGen SourceSection: None
               PythonCallUnary SourceSection: [9,18]`range(10)`
                 CallUnaryMethodNodeGen SourceSection: None
                 ReadNameNodeGen SourceSection: [9,14]`range`

graalpython/com.oracle.graal.python.test/testData/goldenFiles/BasicTests/for09.tast

@@ -49,7 +49,6 @@ ModuleRootNode Name: <module 'for09'> SourceSection: [0,94]`for i in range(1, 10
                       IsBuiltinClassProfile SourceSection: None
                         CachedDispatchFirst SourceSection: None
             GetIteratorExpressionNodeGen SourceSection: [9,21]`range(1, 10)`
-              GetIteratorNodeGen SourceSection: None
               PythonCallBinary SourceSection: [9,21]`range(1, 10)`
                 CallBinaryMethodNodeGen SourceSection: None
                 ReadNameNodeGen SourceSection: [9,14]`range`