[GR-23310] Make test_compile pass.

PullRequest: graalpython/1691

Author: tomasstupka

graalpython/com.oracle.graal.python.test/src/tests/unittest_tags/test_base64.txt

@@ -25,3 +25,4 @@
 *graalpython.lib-python.3.test.test_base64.TestMain.test_decode
 *graalpython.lib-python.3.test.test_base64.TestMain.test_encode_decode
 *graalpython.lib-python.3.test.test_base64.TestMain.test_encode_file
+*graalpython.lib-python.3.test.test_base64.TestMain.test_encode_from_stdin

graalpython/com.oracle.graal.python.test/src/tests/unittest_tags/test_compile.txt

@@ -8,8 +8,10 @@
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_annotation_limit
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_argument_handling
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_argument_order
+*graalpython.lib-python.3.test.test_compile.TestSpecifics.test_compile_ast
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_compiler_recursion_limit
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_dead_blocks_do_not_generate_bytecode
+*graalpython.lib-python.3.test.test_compile.TestSpecifics.test_dict_evaluation_order
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_duplicate_global_local
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_empty
 *graalpython.lib-python.3.test.test_compile.TestSpecifics.test_exec_with_general_mapping_for_locals

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/nodes/literal/DictLiteralNode.java

@@ -104,35 +104,21 @@ protected DynamicDictLiteralNode(ExpressionNode[] keys, ExpressionNode[] values)
             }
         }
 
-        static final class Keys {
-            public final Object[] keys;
-            public final boolean allStrings;
-
-            Keys(Object[] keys, boolean allStrings) {
-                this.keys = keys;
-                this.allStrings = allStrings;
-            }
-        }
-
         @ExplodeLoop
-        private Keys evalKeys(VirtualFrame frame) {
+        private HashingStorage eval(VirtualFrame frame, PythonLanguage lang, ConditionProfile hasFrame) {
             boolean allStrings = true;
             Object[] evalKeys = new Object[this.keys.length];
+            Object[] evalValues = new Object[this.values.length];
             for (int i = 0; i < values.length; i++) {
                 evalKeys[i] = keys[i].execute(frame);
-                if (!(evalKeys[i] instanceof String)) {
+                evalValues[i] = values[i].execute(frame);
+                if (allStrings && !(evalKeys[i] instanceof String)) {
                     allStrings = false;
                 }
             }
-            return new Keys(evalKeys, allStrings);
-        }
-
-        @ExplodeLoop
-        private HashingStorage evalAndSetValues(VirtualFrame frame, HashingStorage dictStorage, Keys evalKeys, ConditionProfile hasFrame) {
-            HashingStorage storage = dictStorage;
+            HashingStorage storage = PDict.createNewStorage(lang, allStrings, evalKeys.length);
             for (int i = 0; i < values.length; i++) {
-                Object val = values[i].execute(frame);
-                storage = libs[i].setItemWithFrame(storage, evalKeys.keys[i], val, hasFrame, frame);
+                storage = libs[i].setItemWithFrame(storage, evalKeys[i], evalValues[i], hasFrame, frame);
             }
             return storage;
         }
@@ -141,9 +127,7 @@ private HashingStorage evalAndSetValues(VirtualFrame frame, HashingStorage dictS
         public PDict create(VirtualFrame frame,
                         @CachedLanguage PythonLanguage lang,
                         @Cached("createBinaryProfile()") ConditionProfile hasFrame) {
-            Keys evalKeys = evalKeys(frame);
-            HashingStorage dictStorage = PDict.createNewStorage(lang, evalKeys.allStrings, evalKeys.keys.length);
-            dictStorage = evalAndSetValues(frame, dictStorage, evalKeys, hasFrame);
+            HashingStorage dictStorage = eval(frame, lang, hasFrame);
             return factory.createDict(dictStorage);
         }
     }

graalpython/lib-python/3/test/test_compile.py

@@ -7,7 +7,7 @@
 import tempfile
 import types
 from test import support
-from test.support import script_helper, FakePath
+from test.support import script_helper, FakePath, impl_detail
 
 class TestSpecifics(unittest.TestCase):
 
@@ -421,6 +421,7 @@ def f():
         self.assertIn("_A__mangled_mod", A.f.__code__.co_varnames)
         self.assertIn("__package__", A.f.__code__.co_varnames)
 
+    @impl_detail("ast module", graalvm=False)
     def test_compile_ast(self):
         fname = __file__
         if fname.lower().endswith('pyc'):