<div dir="ltr"><p class="MsoNormal"><span style="background-image:initial;background-position:initial;background-size:initial;background-repeat:initial;background-origin:initial;background-clip:initial">Hi Vedant,</span><span></span></p><p class="MsoNormal"><br></p><p class="MsoNormal">> <span style="font-size:9.5pt;font-family:Arial,sans-serif;background-image:initial;background-position:initial;background-size:initial;background-repeat:initial;background-origin:initial;background-clip:initial">This sounds fine to me. I don't think we need
to preserve the old single-file behavior. We can add a release note about the
new behavior.</span><span></span></p>

<p class="MsoNormal">Great!<br></p><p class="MsoNormal"><br></p><p class="MsoNormal">I’d like to change the index.html page to only include
coverage data for the filtered functions and do the same for all the ‘source
view’ pages.<span></span></p><p class="MsoNormal"><span> </span></p><p class="MsoNormal">For the index.html page I plan to do the filtering in
`CoverageReport::prepareFileReports`, which will be quite simple.<span></span></p><p class="MsoNormal"><span> </span></p><p class="MsoNormal">Filtering the lines in the source files will be more
complicated, as I will need to find which function a LineSegment comes from. I
think I can do that by getting the function coverage for each filtered function
from the same file, and then checking if the line segment can be found in any
of those. I was wondering if you can think of a better way of mapping from
lines to functions or if you might have something else in mind?<span></span></p><p class="MsoNormal"><span> </span></p><p class="MsoNormal">











</p><p class="MsoNormal">Thanks for your help,<span></span></p><p class="MsoNormal"><span></span></p><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.85px">Sean Eveson</span><br style="color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.85px"><span style="color:rgb(0,0,0);font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.85px">SN Systems - Sony Interactive Entertainment</span><br></div></div></div></div></div>
<br><div class="gmail_quote">On Wed, Sep 20, 2017 at 6:23 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"><div style="word-wrap:break-word"><br><div><span class=""><blockquote type="cite"><div>On Sep 20, 2017, at 7:08 AM, Sean Eveson via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:</div><br class="m_-4854140844903962386Apple-interchange-newline"><div><div dir="ltr"><p class="MsoNormal"><span>Hi All,</span><br></p><div><span> </span><br class="m_-4854140844903962386webkit-block-placeholder"></div><p class="MsoNormal"><span>Normally when produc</span>ing<span> HTML with llvm-cov you get an index.html file and a
directory structure </span>containing<span> .html files
corresponding to source files.<span></span></span></p><div><span> </span><br class="m_-4854140844903962386webkit-block-placeholder"></div><p class="MsoNormal">Currently w<span>hen </span>using any of
the -name*= command line options with -format=html, <span>the
output is a single file (functions.html) containing the coverage for those
specific functions.<span></span></span></p><div><span> </span><br class="m_-4854140844903962386webkit-block-placeholder"></div><p class="MsoNormal"><span>Having a single file is not as
useful as the full </span><span>directory structure </span><span>when you have a large number of functions in your
filter (using -name-regex or -name-whitelist). </span>Could we<span> put the single file HTML output behind an option and
produce the full structure by default</span><span style="color:red"> </span>when
using the -name* options<span>.</span></p></div></div></blockquote></span><div>This sounds fine to me. I don't think we need to preserve the old single-file behavior. We can add a release note about the new behavior.</div><div><br></div><div>vedant</div><blockquote type="cite"><div><span class=""><div dir="ltr"><p class="MsoNormal"><span><span></span></span></p><div><span> </span><br class="m_-4854140844903962386webkit-block-placeholder"></div><p class="MsoNormal"><span>What do people think?<span></span></span></p><div><span> </span><br class="m_-4854140844903962386webkit-block-placeholder"></div><p class="MsoNormal"><span>Thanks,<span></span></span></p><p class="MsoNormal"><span><br></span></p><div><div class="m_-4854140844903962386gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.85px">Sean Eveson</span><br style="font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.85px"><span style="font-family:'Segoe UI','Segoe UI Web Regular','Segoe UI Symbol','Helvetica Neue',Helvetica,Arial,sans-serif;font-size:13px;line-height:18.85px">SN Systems - Sony Interactive Entertainment</span><br></div></div></div></div></div>
</div></span>
______________________________<wbr>_________________<br>LLVM Developers mailing list<br><a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br></div></blockquote></div><br></div></blockquote></div><br></div></div>