<div dir="ltr"><p class="MsoNormal"><span style="font-family:"calibri light",sans-serif">Hi Vedant,</span></p><p class="MsoNormal"><span style="font-family:"calibri light",sans-serif"><br></span></p>

<p class="MsoNormal"><span style="font-family:"calibri light",sans-serif">I have
attached the index.html page which includes the project directory in the
summary, and separate columns for files and directories in the index table. Listing
the project directory in summary allows user to construct the full file path.</span></p><p class="MsoNormal"><span style="font-family:"calibri light",sans-serif"><br></span></p>

<p class="MsoNormal"><span style="font-family:"calibri light",sans-serif">For example:
</span></p>

<pre><span style="font-size:11pt;font-family:"calibri light",sans-serif">Project directory: I:\svn\PS4\toolchain-test\tests\branches\PS4_4.500\benchmarks</span></pre><pre><span style="font-size:11pt;font-family:"calibri light",sans-serif">File directory: \Bullet-2.76\Demos\Benchmarks</span></pre><pre><span style="font-size:11pt;font-family:"calibri light",sans-serif"> The file <a>BenchmarkDemo.cpp</a> could find in the folder:<br> I:\svn\PS4\toolchain-test\tests\branches\PS4_4.500\benchmarks\Bullet-2.76\Demos\Benchmarks .<br><br></span></pre>

<p class="MsoNormal"><span style="font-family:"calibri light",sans-serif">Thanks,</span></p>

<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 20, 2016 at 10:03 PM, Vedant Kumar <span dir="ltr"><<a href="mailto:vsk@apple.com" target="_blank">vsk@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">vsk added a comment.<br>
<span class=""><br>
> If the project is branched, the longest-common prefix of the path (I called the project directory) will show the branch name. The project directory will tell the user where the source code come from.<br>
<br>
<br>
</span>It would be easier for me to understand the motivation behind this if you could upload an example report with your changes. I still don't understand what it is about the current scheme that makes it difficult for readers to figure out where some source file comes from.<br>
<br>
<br>
================<br>
Comment at: tools/llvm-cov/<wbr>SourceCoverageViewHTML.cpp:431<br>
@@ -390,1 +430,3 @@<br>
+  OSRef << BeginCenteredDiv << BeginTable;<br>
+  emitColumnLabelsForIndex(<wbr>OSRef);<br>
   for (unsigned I = 0, E = FileReports.size(); I < E; ++I)<br>
----------------<br>
This patch still seems larger than it needs to be. Can't we get the entire desired functional change by just inserting another `emitFileSummary(OSRef, "Totals", Totals, /*IsTotals=*/true)` here?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24687" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24687</a><br>
<br>
<br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><font size="2"><span style="font-family:arial,helvetica,sans-serif"></span></font><font face="Calibri" size="2"><span style="font-size:11pt"><font size="2"><span style="font-size:10pt"><font color="#1F497D" face="Arial">Ying Yi<br>SN Systems - Sony Interactive Entertainment</font></span></font></span></font><br></div></div></div></div></div></div></div></div>
</div>