[cfe-commits] r58100 - in /cfe/trunk: lib/Sema/SemaDecl.cpp test/SemaCXX/fntype-decl.cpp

[Argiris Kirtzidis]

Doug Gregor wrote:
>  
> Thanks for the diagnosis; I'll get this fixed. It sure would have been
> nice if it crashed on Mac OS :)
>   

Here's something to help you on the crash department.
The attached patch modifies llvm::MallocAllocator and gets it to fill 
the object with garbage before free'ing it.
Try applying it and running the tests.
This is what the MS debug CRT does automatically and it works great for 
catching this kind of bugs.

Is it reasonable to add something like this patch on llvm and have it 
enabled for debug builds ?

> It does seem odd that we're destroying the TypedefDecl before
> destroying the FunctionDecl but I'll look into it.
>   

TranslationUnit destroys the decls at the order of  receiving them, 
should that be reverse order ?

-Argiris
-------------- next part --------------
A non-text attachment was scrubbed...
Name: alloc.patch
Type: text/x-diff
Size: 1093 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20081025/73ef6df4/attachment.patch>