<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On May 26, 2008, at 11:55 PM, Chandler Carruth wrote:</div><blockquote type="cite"><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">This would make the summary of<br> <<a href="http://chandlerc.net/llvm-coverage/lib/Target/PowerPC/" target="_blank">http://chandlerc.net/llvm-coverage/lib/Target/PowerPC/</a><br> PPCHazardRecognizers.cpp.gcov.html><br> much more favorable.</blockquote><div><br>I'm not seeing the bad aspect of this particular file? It has pretty good coverage, is code calling into this "bad"? If so, then assertions, or logging or something might be more appropriate. Coverage just says that it got used, not that its use was "correct". ;] <br></div></div></blockquote></div><br><div>I think he means the lines like:</div><div><br></div><div><pre><font class="Apple-style-span" face="Helvetica" size="3"><span class="Apple-style-span" style="font-size: 12px; white-space: normal;"> switch (Opcode) {</span></font></pre><pre><font class="Apple-style-span" face="Helvetica" size="3"><span class="Apple-style-span" style="font-size: 12px; white-space: normal;">default: assert(0 && "Unknown load!");</span></font></pre><pre><br></pre><pre>where the default case is never executed.</pre><pre><font class="Apple-style-span" face="Helvetica" size="3"><span class="Apple-style-span" style="font-size: 12px; white-space: normal;">-Chris</span></font></pre></div></body></html>