[pypy-commit] pypy default: Move gc_stack_bottom before the increment of stacks_counter. This is needed because debug builds check the value of stack_counter and expect the old one.

[arigo]

Author: Armin Rigo <arigo at tunes.org>
Branch: 
Changeset: r68840:7cb3f52582ea
Date: 2014-01-22 12:18 +0100
http://bitbucket.org/pypy/pypy/changeset/7cb3f52582ea/

Log:	Move gc_stack_bottom before the increment of stacks_counter. This
	is needed because debug builds check the value of stack_counter and
	expect the old one.

diff --git a/pypy/module/cpyext/api.py b/pypy/module/cpyext/api.py
--- a/pypy/module/cpyext/api.py
+++ b/pypy/module/cpyext/api.py
@@ -600,8 +600,8 @@
         from pypy.module.cpyext.pyobject import Reference
         # we hope that malloc removal removes the newtuple() that is
         # inserted exactly here by the varargs specializer
+        llop.gc_stack_bottom(lltype.Void)   # marker for trackgcroot.py
         rffi.stackcounter.stacks_counter += 1
-        llop.gc_stack_bottom(lltype.Void)   # marker for trackgcroot.py
         retval = fatal_value
         boxed_args = ()
         try: