<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 9, 2016 at 8:23 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.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=""><p dir="ltr"><br>
On Mar 9, 2016 8:18 AM, "Teresa Johnson" <<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>> wrote:<br>
><br>
><br>
><br>
> On Wed, Mar 9, 2016 at 8:13 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br>
>><br>
>><br>
>> On Mar 9, 2016 8:11 AM, "Teresa Johnson via llvm-commits" <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
>> ><br>
>> > tejohnson added a subscriber: tejohnson.<br>
>> > tejohnson added a comment.<br>
>> ><br>
>> > Thanks for doing this!<br>
>> ><br>
>> > I will fix these 3, which one of my changes would have provoked. It is a benign case since we do have an assert at the use site checking the same condition that the inits are guarded with, but there is no reason not to 0 init these to avoid the bogus warning.<br>
>><br>
>> The reason not to is that it would silence msan catching the unintentional codepath (eg: if you didn't have the assert there, and they init and use conditions didn't match, msan would tell us - if we add what we believe to be a dead init, then msan no longer helps us here)<br>
><br>
><br>
> Hmm, ok. It's unfortunate that the warning and msan are at odds, although I don't know how to reconcile them. </p>
</span><p dir="ltr">The approach we've taken so far, so far as I'm aware, is to disable warnings like this that have such false positives.</p><span class="">
<p dir="ltr">> Will leave as is then. Is it worth a comment indicating why we are leaving them uninitialized?</p>
</span><p dir="ltr">Not sure. If you think it helps readability. If it's fairly clear the variable is only used under the same condition that it's initialized, I think you could leave it.</p></blockquote><div><br></div><div>Ok, sounds good.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<p dir="ltr">>><br>
>> ><br>
>> > Warning C4701   potentially uninitialized local variable 'FnEntry8BitAbbrev' used       LLVMBitWriter   c:\llvm\llvm\lib\bitcode\writer\bitcodewriter.cpp       2353<br>
>> > Warning C4701   potentially uninitialized local variable 'FnEntry7BitAbbrev' used       LLVMBitWriter   c:\llvm\llvm\lib\bitcode\writer\bitcodewriter.cpp       2357<br>
>> > Warning C4701   potentially uninitialized local variable 'FnEntry6BitAbbrev' used       LLVMBitWriter   c:\llvm\llvm\lib\bitcode\writer\bitcodewriter.cpp       2355<br>
>> ><br>
>> ><br>
>> > <a href="http://reviews.llvm.org/D17983" target="_blank">http://reviews.llvm.org/D17983</a><br>
>> ><br>
>> ><br>
>> ><br>
>> > _______________________________________________<br>
>> > llvm-commits mailing list<br>
>> > <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
>> > <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
><br>
><br>
><br>
><br>
> -- <br>
> Teresa Johnson |<br>
>  Software Engineer |<br>
>  <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |<br>
>  <a href="tel:408-460-2413" value="+14084602413" target="_blank">408-460-2413</a></p>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 408-460-2413</td></tr></tbody></table></span></div>
</div></div>