<div dir="ltr"><div>I agree, checked in binaries are not always pretty. But some coverage depends checked in binaries (or at very least is dramatically harder to get the same thing from source)</div><div><br></div><div>Are we saying that sacrificing coverage to keep tests smaller should be the default trade off?</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 8, 2018 at 10:04 AM, Zachary Turner via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zturner added a comment.<br>
<span class=""><br>
In <a href="https://reviews.llvm.org/D47708#1126669" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D47708#1126669</a>, @lemo wrote:<br>
<br>
> Doesn't the LIT based test drop the split-function case (originally<br>
>  produced with PGO)?<br>
><br>
> Sorry for being late to the party, but it seems beneficial to have both LIT<br>
>  *and* checked in binaries since in general they are complementary: checking<br>
>  against freshly built binaries only covers a matching set of toolchain<br>
>  components (in particular it's hard to cover the cross-targeting scenarios).<br>
><br>
> Other than the inconvenience with Phabricator, is there a reason not to<br>
>  include the original tests as well? The size of the binaries?<br>
<br>
<br>
</span>At least when it comes to checked in executables, we trigger virus scanners sometimes which is pretty annoying.  I don't mind a checked in PDB, but they get pretty big sometimes and you have to go out of your way to make them small by specifying things like `/nodefaultlib`.  But if you can get PDBs below about 200k, then checking them in might not be so bad.  But I'd like to avoid checking in executables.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D47708" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D47708</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>