<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 9, 2016 at 8:13 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: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.</p>
</span><p dir="ltr">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)</p></blockquote><div><br></div><div>Hmm, ok. It's unfortunate that the warning and msan are at odds, although I don't know how to reconcile them. Will leave as is then. Is it worth a comment indicating why we are leaving them uninitialized?</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p dir="ltr"><span class="">><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></span>
> _______________________________________________<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>
</p>
</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>