[PATCH] Poison all slabs on BumpPtrAllocator::Reset

Pete Cooper via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 8 17:15:49 PST 2016


Thanks for the CC Alexey.

LGTM.  I don’t know how I missed this case.  I’ll apply it now and commit if it passes.  Thanks b17 c0de.

Pete
> On Jan 8, 2016, at 5:09 PM, Alexey Samsonov <vonosmas at gmail.com> wrote:
> 
> +Pete, Justin who landed the original poisoning for BumpPtrAllocator.
> 
> On Fri, Jan 8, 2016 at 3:44 PM, David Blaikie <dblaikie at gmail.com <mailto:dblaikie at gmail.com>> wrote:
> bump/pass
> 
> Alexey - is this the right asanification here?
> 
> On Fri, Dec 18, 2015 at 1:07 AM, b17 c0de via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
> BumpPtrAllocator::Reset should also poison the first slab which doesn't get deallocated. Please apply.
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
> 
> 
> 
> 
> 
> -- 
> Alexey Samsonov
> vonosmas at gmail.com <mailto:vonosmas at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160108/89d2a02d/attachment.html>


More information about the llvm-commits mailing list