<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 13, 2016 at 10:25 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">I'm sorry, what is this data comparing? Specifically, what is extended in clang? Is it just this patch applied?</div></blockquote><div><br></div><div><div style="font-size:12.8px">Yes, what is extended in clang is just this patch applied (i.e., anything named "opt*").</div><div style="font-size:12.8px">This is comparing build times, execution times, and statistical data from "-mllvm -stats" between the version of clang with this patch applied and a version of clang without the patch (i.e., anything named "cur*).</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">These comparison data are respectively in the sheets named "Build Clang", "Run Clang" and "Stats [*]".</div><div style="font-size:12.8px">Just in case, I also generated opt*- and cur*- clang binaries using both clang and gcc.</div><div style="font-size:12.8px">The sheets named "Stats [Clang]" and "Stats [GCC]" both represent statistical data with clang build clang and gcc.</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">For run times, I simply ran regression tests. There may be other ways to get better runtime measurements (?). But I thought regression tests could give us something to start with.</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">I also put some descriptions under the "Legend" sheet. I recognize that this could be clearer. So, please don't hesitate to ask me for any clarification.</div></div><div style="font-size:12.8px"><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br><div class="gmail_quote"><div><div class="h5"><div dir="ltr">On Wed, Jan 13, 2016 at 9:27 PM Larisse Voufo via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div class="h5">lvoufo added a comment.<br>
<br>
Alas, I wanted to address all these comments today, but I ended up being consumed with a bit of data crunching, while building clang with different versions of clang (with and without this patch).<br>
For what it's worth---and my intention was for this to help this ongoing discussion, here are the results:<br>
<a href="https://docs.google.com/spreadsheets/d/19W1167l9QFrXMXccX4-cValmC5EtlFfUbT7Tr3xaoJs/edit#gid=1063925865usp=sharing" rel="noreferrer" target="_blank">https://docs.google.com/spreadsheets/d/19W1167l9QFrXMXccX4-cValmC5EtlFfUbT7Tr3xaoJs/edit#gid=1063925865usp=sharing</a><br>
<br>
More load instructions are deleted as expected, a little bit above 9% increase, and there are other gains and losses (highlighted in the spreadsheet) which I haven't gotten around to analyzing yet.<br>
In general, there is either a slight compilation time increase, below 1%, when there is no decrease.<br>
And there is a runtime improvement of 4% with clang binaries build with GCC.<br>
Unexpectedly, there is a small runtime hit, below 0/5%, with clang binaries built with clang.<br>
<br>
The times here were collected while other programs may have been running on my machine. I am going to rerun the scripts overnight and see if we get different outcomes in tomorrow...<br>
<br>
In the meantime, I welcome any suggestion for better data crunching...<br>
<br>
I will also be addressing all the above comments tomorrow.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D15124" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15124</a><br>
<br>
<br>
<br></div></div>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>
</blockquote></div><br></div></div>