LGTM, but the error message of "unexpected source file" isn't very clear. Maybe something more along the lines of "multiple sources for a single basic block" would be more precise?<br><br>On Tuesday, November 5, 2013, Yuchen Wu  wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">llvm-cov: Removed StringMap holding GCOVLines.<br>
<br>
According to the hazy gcov documentation, it appeared to be technically<br>
possible for lines within a block to belong to different source files.<br>
However, upon further investigation, gcov does not actually support<br>
multiple source files for a single block.<br>
<br>
This change removes a level of separation between blocks and lines by<br>
replacing the StringMap of GCOVLines with a SmallVector of ints<br>
representing line numbers. This also means that the GCOVLines class is<br>
no longer needed.<br>
<br>
This paves the way for supporting the "-a" option, which will output<br>
block information.                                        </blockquote>