r198885 - Revert "Disable LeakSanitizer in TableGen binaries, see PR18325"

Alp Toker alp at nuanti.com
Thu Jan 9 16:40:33 PST 2014


On 09/01/2014 22:48, Chandler Carruth wrote:
>
> On Thu, Jan 9, 2014 at 2:41 PM, Richard Smith <richard at metafoo.co.uk 
> <mailto:richard at metafoo.co.uk>> wrote:
>
>     On Thu, Jan 9, 2014 at 2:29 PM, Aaron Ballman
>     <aaron at aaronballman.com <mailto:aaron at aaronballman.com>> wrote:
>
>         If we're going to back out the revert, can we put the code into an
>         #ifdef so that the reserved namespace identifier is protected when
>         compiling with something that doesn't understand lsan? Then we can
>         argue over the "right" way with some protection.
>
>
>     I'm not opposed to that, if we have a suitable predefine. But we
>     already have *loads* of code in Clang that defines identifiers in
>     the reserved namespace (try grepping for '[A-Za-z]__[A-Za-z]' in
>     include/ to find a bunch of them), and none of our supported C++
>     implementations (for clang 3.5) have a problem with this, so I
>     don't see that there's a lot of value in doing so.
>
>
> Further, I don't think we should slow down the efforts to get LSan 
> bootstrapping effectively while we figure out the correct predefine -- 
> we can add one later as the discussion converges. That had been my 
> plan from the beginning.

Hi Chandler,

This is sounding positive.

What's the schedule?

Alp.


>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list