|
From: Nicholas N. <n.n...@gm...> - 2009-08-11 23:28:02
|
On Wed, Aug 12, 2009 at 2:18 AM, Josef Weidendorfer<Jos...@gm...> wrote: > > I reopened 197988, as this is still valid. The easiest is > to enlarge the tool's stack to a 2 MB. Or is there some adress space > problem for 32bit here? I don't know. Increasing to 2MB will reduce the problem but not solve it fully -- someone somewhere someday will use Boost to generate a symbol that still overflows it. If adding a recursion limit is easy, I'd like to do that as it will definitely avoid crashes. (Ideally the symbol would be partially demangled so the user can see something useful.) Later on, if the stack size is increased then the recursion limit can also be increased, which would reduce the likelihood of it occurring. > It would be cool to have VG_(discard_translations) in the tool interface > to get rid of an ugly hack (calling a non-public function from core) - I > sent a proposed patch to Julian... Can you post the patch here? The change sounds ok to me in principle, but I think you'll have to create a new file include/pub_tool_transtab.h so I'd like to see the patch. Thanks. Nick |