<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 17, 2015 at 11:16 PM, Dmitry Vyukov <span dir="ltr"><<a href="mailto:dvyukov@google.com" target="_blank">dvyukov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Feb 17, 2015 at 9:57 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
><br>
><br>
> On Mon, Feb 16, 2015 at 5:53 AM, Dmitry Vyukov <<a href="mailto:dvyukov@google.com">dvyukov@google.com</a>> wrote:<br>
>><br>
>> Author: dvyukov<br>
>> Date: Mon Feb 16 07:53:00 2015<br>
>> New Revision: 229392<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=229392&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=229392&view=rev</a><br>
>> Log:<br>
>> tsan: fix compiler warning<br>
>><br>
>> mingw gcc complains:<br>
>> warning: 'err' may be used uninitialized in this function<br>
>> [-Wmaybe-uninitialized]<br>
>>        Printf("Failed to read options from '%s': error %d\n", value, err);<br>
><br>
><br>
> Was the warning correct? Or could we be sure that ReadFileToBuffer would<br>
> initialize 'err'?<br>
><br>
> Generally we should try not to unnecessarily initialize variables just to<br>
> satisfy warnings because it hurts sanitizers (though I suppose that argument<br>
> doesn't apply to the sanitizer libraries themselves - it still seems like a<br>
> good principle to apply)<br>
<br>
</span>I guess it is as incorrect as most of C++ compiler warnings.<br>
<br>
This happened on windows, where you usually don't have a great choice<br>
of host compilers. So I am not sure how else we can resolve it.<br></blockquote><div><br>We simply disable the warnings, don't have to switch host compilers - we have a bunch of MSVC warnings disabled in our build configs already, I believe.<br> </div></div><br></div></div>