<div dir="ltr">Doesn't the LIT based test drop the split-function case (originally produced with PGO)?<div><br></div><div>Sorry for being late to the party, but it seems beneficial to have both LIT *and* checked in binaries since in general they are complementary: checking against freshly built binaries only covers a matching set of toolchain components (in particular it's hard to cover the cross-targeting scenarios).</div><div><br></div><div>Other than the inconvenience with Phabricator, is there a reason not to include the original tests as well? The size of the binaries?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 8, 2018 at 1:27 AM, Pavel Labath via Phabricator via lldb-commits <span dir="ltr"><<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">labath added a comment.<br>
<br>
Thank you for your patience. I am very happy with the final result here.<br>
<br>
I personally like to keep number of files in a test minimal (hence I inlined the test into the cpp file), but overall that's not really important.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="https://reviews.llvm.org/D47708" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D47708</a><br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/lldb-commits</a><br>
</div></div></blockquote></div><br></div>