[LLVMdev] DejaGnu test-suite coverage

[Chris Lattner]

On May 26, 2008, at 11:55 PM, Chandler Carruth wrote:
> This would make the summary of
> <http://chandlerc.net/llvm-coverage/lib/Target/PowerPC/
> PPCHazardRecognizers.cpp.gcov.html>
> much more favorable.
>
> 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". ;]

I think he means the lines like:

switch (Opcode) {
default: assert(0 && "Unknown load!");

where the default case is never executed.
-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20080527/96d6be36/attachment.html>