since PyNumber_Float is now called directly and not through the landing func, we need to convert the NULL result to Sulong, too.

Author: timfel

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

@@ -2390,11 +2390,15 @@ Object doGeneric(@SuppressWarnings("unused") Object module, Object object) {
                 CompilerDirectives.transferToInterpreterAndInvalidate();
                 floatNode = insert(BuiltinConstructorsFactory.FloatNodeFactory.create(null));
             }
+            return getToSulongNode().execute(floatNode.executeWith(PythonBuiltinClassType.PFloat, asPythonObjectNode.execute(object)));
+        }
+
+        private CExtNodes.ToSulongNode getToSulongNode() {
             if (toSulongNode == null) {
                 CompilerDirectives.transferToInterpreterAndInvalidate();
                 toSulongNode = insert(CExtNodes.ToSulongNode.create());
             }
-            return toSulongNode.execute(floatNode.executeWith(PythonBuiltinClassType.PFloat, asPythonObjectNode.execute(object)));
+            return toSulongNode;
         }
 
         @Specialization(replaces = "doGeneric")
@@ -2403,7 +2407,7 @@ Object doGenericErr(Object module, Object object) {
                 return doGeneric(module, object);
             } catch (PException e) {
                 transformToNative(e);
-                return getNativeNull(module);
+                return getToSulongNode().execute(getNativeNull(module));
             }
         }
     }