<div dir="ltr">valgrind is satisfied. thanks! <div>msan bot will need some time to become green. </div><div><br></div><div>--kcc </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 1, 2014 at 4:51 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, Aug 1, 2014 at 8:46 AM, Kostya Serebryany <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br>

><br>
><br>
><br>
> On Fri, Aug 1, 2014 at 4:42 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> On Fri, Aug 1, 2014 at 8:38 AM, Kostya Serebryany <<a href="mailto:kcc@google.com">kcc@google.com</a>> wrote:<br>
>> >><br>
>> >> I don't have access to either a valgrind or a msan build, but I took a<br>
>> ><br>
>> > For msan you do need a separate build, but for valgrind your just run<br>
>> > valgrind on a regular build.<br>
>><br>
>> Valgrind doesn't work on Windows; I'm uncertain whether msan does or not?<br>
><br>
> Nope. On Windows  your are on your own against uninits (unless you want to<br>
> try DrMemory)<br>
>><br>
>><br>
>> >><br>
>> >> stab in the dark with r214502. I'll see if the msan bot thinks I've<br>
>> ><br>
>> > Nope, the report is still there.<br>
>><br>
>> Drat! I'll do a bit more investigating...<br>
><br>
><br>
> valgrind with --track-origins=yes gives this:<br>
> ==3308==  Uninitialised value was created by a heap allocation<br>
> ==3308==    at 0x402F7C4: malloc<br>
> (valgrind/coregrind/m_replacemalloc/vg_replace_malloc.c:270)<br>
> ==3308==    by 0x25CEAEA: clang::Preprocessor::AllocateMacroInfo()<br>
> ==3308==    by 0x25CEC3C:<br>
> clang::Preprocessor::AllocateMacroInfo(clang::SourceLocation)<br>
> ==3308==    by 0x25D312A:<br>
> clang::Preprocessor::HandleDefineDirective(clang::Token&, bool)<br>
> ==3308==    by 0x25D8E58:<br>
> clang::Preprocessor::HandleDirective(clang::Token&)<br>
> ...<br>
<br>
</div></div>Yes, I found another code path where this wasn't getting initialized.<br>
Perhaps 214504 fixes this?<br>
<br>
(Thank you for your help with tracking this down!)<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>