|
From: <sv...@va...> - 2013-03-10 12:51:25
|
bart 2013-03-10 12:51:15 +0000 (Sun, 10 Mar 2013)
New Revision: 13320
Log:
Darwin: Make stack growth tracking consistent with other architectures
Modified files:
trunk/coregrind/m_sigframe/sigframe-amd64-darwin.c
Modified: trunk/coregrind/m_sigframe/sigframe-amd64-darwin.c (+2 -8)
===================================================================
--- trunk/coregrind/m_sigframe/sigframe-amd64-darwin.c 2013-03-10 12:49:08 +00:00 (rev 13319)
+++ trunk/coregrind/m_sigframe/sigframe-amd64-darwin.c 2013-03-10 12:51:15 +00:00 (rev 13320)
@@ -89,14 +89,8 @@
static Bool extend ( ThreadState *tst, Addr addr, SizeT size )
{
ThreadId tid = tst->tid;
- /* For tracking memory events, indicate the entire frame has been
- allocated. Except, don't mess with the area which
- overlaps the previous frame's redzone. */
- /* XXX is the following call really right? compared with the
- amd64-linux version, this doesn't appear to handle the redzone
- in the same way. */
- VG_TRACK( new_mem_stack_signal,
- addr - VG_STACK_REDZONE_SZB, size, tid );
+ VG_TRACK( new_mem_stack_signal, addr - VG_STACK_REDZONE_SZB,
+ size + VG_STACK_REDZONE_SZB, tid );
return True;
}
|