Author: Armin Rigo <[email protected]>
Branch: 
Changeset: r50695:fb6b04734b48
Date: 2011-12-19 07:32 +0100
http://bitbucket.org/pypy/pypy/changeset/fb6b04734b48/

Log:    don't overwrite a previous exception with MemoryError, if any

diff --git a/pypy/jit/backend/llsupport/llmodel.py 
b/pypy/jit/backend/llsupport/llmodel.py
--- a/pypy/jit/backend/llsupport/llmodel.py
+++ b/pypy/jit/backend/llsupport/llmodel.py
@@ -109,7 +109,8 @@
 
         def save_exception_memoryerr():
             save_exception()
-            self.saved_exc_value = "memoryerror!"    # for tests
+            if not self.saved_exc_value:
+                self.saved_exc_value = "memoryerror!"    # for tests
 
         self.pos_exception = pos_exception
         self.pos_exc_value = pos_exc_value
@@ -142,9 +143,11 @@
         def save_exception_memoryerr():
             from pypy.rpython.annlowlevel import cast_instance_to_base_ptr
             save_exception()
-            exc = MemoryError()
-            exc = cast_instance_to_base_ptr(exc)
-            self.saved_exc_value = lltype.cast_opaque_ptr(llmemory.GCREF, exc)
+            if not self.saved_exc_value:
+                exc = MemoryError()
+                exc = cast_instance_to_base_ptr(exc)
+                exc = lltype.cast_opaque_ptr(llmemory.GCREF, exc)
+                self.saved_exc_value = exc
 
         from pypy.rlib import rstack
         STACK_CHECK_SLOWPATH = lltype.Ptr(lltype.FuncType([lltype.Signed],
@@ -193,8 +196,9 @@
 
     _ON_JIT_LEAVE_FUNC = lltype.Ptr(lltype.FuncType([], lltype.Void))
 
-    def get_on_leave_jitted_int(self, save_exception, memoryerror=False):
-        if memoryerror:
+    def get_on_leave_jitted_int(self, save_exception,
+                                default_to_memoryerror=False):
+        if default_to_memoryerror:
             f = llhelper(self._ON_JIT_LEAVE_FUNC, 
self.on_leave_jitted_memoryerr)
         elif save_exception:
             f = llhelper(self._ON_JIT_LEAVE_FUNC, 
self.on_leave_jitted_save_exc)
diff --git a/pypy/jit/backend/x86/assembler.py 
b/pypy/jit/backend/x86/assembler.py
--- a/pypy/jit/backend/x86/assembler.py
+++ b/pypy/jit/backend/x86/assembler.py
@@ -276,7 +276,7 @@
         self.mc = codebuf.MachineCodeBlockWrapper()
         # call on_leave_jitted_save_exc()
         addr = self.cpu.get_on_leave_jitted_int(save_exception=True,
-                                                memoryerror=True)
+                                                default_to_memoryerror=True)
         self.mc.CALL(imm(addr))
         self.mc.MOV_ri(eax.value, self.cpu.propagate_exception_v)
         self._call_footer()
_______________________________________________
pypy-commit mailing list
[email protected]
http://mail.python.org/mailman/listinfo/pypy-commit

Reply via email to