dir builtin: sort returned list of keys

Author: cosminbasca

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/BuiltinFunctions.java

@@ -101,6 +101,8 @@
 import com.oracle.graal.python.builtins.objects.function.PArguments;
 import com.oracle.graal.python.builtins.objects.function.PKeyword;
 import com.oracle.graal.python.builtins.objects.ints.PInt;
+import com.oracle.graal.python.builtins.objects.list.ListBuiltins;
+import com.oracle.graal.python.builtins.objects.list.PList;
 import com.oracle.graal.python.builtins.objects.module.PythonModule;
 import com.oracle.graal.python.builtins.objects.object.PythonObject;
 import com.oracle.graal.python.builtins.objects.object.PythonObjectLibrary;
@@ -484,11 +486,14 @@ Object locals(VirtualFrame frame, @SuppressWarnings("unused") Object object,
                         @Cached MaterializeFrameNode materializeNode,
                         @Cached("createBinaryProfile()") ConditionProfile inGenerator,
                         @Cached("create(KEYS)") LookupAndCallUnaryNode callKeysNode,
+                        @Cached ListBuiltins.ListSortNode sortNode,
                         @Cached ListNodes.ConstructListNode constructListNode) {
 
             Object localsDict = LocalsNode.getLocalsDict(frame, this, readLocalsNode, readCallerFrameNode, materializeNode, inGenerator);
             Object keysObj = callKeysNode.executeObject(frame, localsDict);
-            return constructListNode.execute(keysObj);
+            PList list = constructListNode.execute(keysObj);
+            sortNode.sort(frame, list);
+            return list;
         }
 
         @Specialization(guards = "!isNoValue(object)")

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/list/ListBuiltins.java

@@ -877,6 +877,12 @@ protected static boolean maySideEffect(PList list, PKeyword[] keywords) {
             return false;
         }
 
+        public final Object sort(VirtualFrame frame, PList list) {
+            return this.execute(frame, list, PArguments.EMPTY_VARARGS, PKeyword.EMPTY_KEYWORDS);
+        }
+
+        public abstract Object execute(VirtualFrame frame, PList list, Object[] arguments, PKeyword[] keywords);
+
         @Specialization(guards = "!isSortable(list, lenNode)")
         @SuppressWarnings("unused")
         Object none(VirtualFrame frame, PList list, Object[] arguments, PKeyword[] keywords,
@@ -907,6 +913,10 @@ Object defaultSort(VirtualFrame frame, PList list, Object[] arguments, PKeyword[
             callSort.execute(sortMethod, arguments, keywords);
             return PNone.NONE;
         }
+
+        public static ListSortNode create() {
+            return ListBuiltinsFactory.ListSortNodeFactory.create();
+        }
     }
 
     @Builtin(name = __LEN__, minNumOfPositionalArgs = 1)