r230160 - Relax the requirement on sized deallocation a bit: Default on unsized delete if sized delete is not provided in global scope, and -fdefine-sized-deallocation option is disabled.

Larisse Voufo lvoufo at google.com
Mon Feb 23 11:04:46 PST 2015


On Mon, Feb 23, 2015 at 1:29 AM, Renato Golin <renato.golin at linaro.org>
wrote:

> On 22 February 2015 at 06:59, Reid Kleckner <rnk at google.com> wrote:
> > Nice, that seems like a reasonable expedient compromise. :)
>
> Unfortunately, that broke our sanitizer bot:
>
> http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/builds/3859
>
> After cleaning the old failure, it still fails now:
>
> http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15-full/builds/3879
>
> Can you have a look at that, please?
>

I am not convinced that this failure has anything to do with this patch in
particular.
I have ran into a number of new/delete mismatches upon enabling
sized-deallocation (which I am investigating).
So, I don't think that this is a bug introduced by this patch so much as it
is a bug in the sanitizer.



>
> thanks,
> --renato
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150223/a4b28a78/attachment.html>


More information about the cfe-commits mailing list