last call argument doesn't need to be simple, so don't force it into a temp

diff --git a/Cython/Compiler/ExprNodes.py b/Cython/Compiler/ExprNodes.py
index 4384c8a418a139b1e16b9852f1654e848f38da17..f4d3f9725b56905f1b9dfc87ffd11a616c65a352 100755 (executable)
--- a/Cython/Compiler/ExprNodes.py
+++ b/Cython/Compiler/ExprNodes.py
@@ -2966,9 +2966,13 @@ class SimpleCallNode(CallNode):
             self.is_temp = 1
         # Coerce arguments
         some_args_in_temps = False
-        for i in range(min(max_nargs, actual_nargs)):
+        max_fitting_arg_count = min(max_nargs, actual_nargs)
+        for i in range(max_fitting_arg_count):
             formal_type = func_type.args[i].type
-            arg = self.args[i].coerce_to(formal_type, env).coerce_to_simple(env)
+            arg = self.args[i].coerce_to(formal_type, env)
+            if i < max_fitting_arg_count-1:
+                # last argument will be evaluated last either way
+                arg = arg.coerce_to_simple(env)
             if arg.is_temp:
                 if i > 0: # first argument doesn't matter
                     some_args_in_temps = True
@@ -2992,7 +2996,7 @@ class SimpleCallNode(CallNode):
             # if some args are temps and others are not, they may get
             # constructed in the wrong order (temps first) => make
             # sure they are either all temps or all not temps
-            for i in range(min(max_nargs, actual_nargs)-1):
+            for i in range(max_fitting_arg_count-1):
                 if i == 0 and self.self is not None:
                     continue # self is ok
                 arg = self.args[i]