Pass through cls when constructing no-argument set

Zach Olstein · Zach Olstein · commit d0b0c1ed530a · 2020-01-07T15:55:26.000-08:00
Adds cls as an argument to factory.createSet in the no-argument
specialization of the set constructor. Without this, attempts to
instantiate a subclass of set with no arguments will instead create a
regular set.
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/set/SetNodes.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/set/SetNodes.java
@@ -93,7 +93,7 @@ PSet setString(VirtualFrame frame, LazyPythonClass cls, String arg,
         @SuppressWarnings("unused")
         PSet set(LazyPythonClass cls, PNone none,
                         @Shared("factory") @Cached PythonObjectFactory factory) {
-            return factory.createSet();
+            return factory.createSet(cls);
         }
 
         @Specialization(guards = "!isNoValue(iterable)")

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,7 @@ PSet setString(VirtualFrame frame, LazyPythonClass cls, String arg,`
`93`	`93`	`@SuppressWarnings("unused")`
`94`	`94`	`PSet set(LazyPythonClass cls, PNone none,`
`95`	`95`	`@Shared("factory") @Cached PythonObjectFactory factory) {`
`96`		`- return factory.createSet();`
	`96`	`+ return factory.createSet(cls);`
`97`	`97`	`}`
`98`	`98`
`99`	`99`	`@Specialization(guards = "!isNoValue(iterable)")`