<div dir="auto"><br><div class="gmail_extra" dir="auto"><br><div class="gmail_quote">On Oct 4, 2017 3:07 PM, "Michael Zolotukhin via Phabricator via llvm-commits" <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br type="attribution"><blockquote class="m_6651743417152066690quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mzolotukhin added a comment.<br>
<div class="m_6651743417152066690quoted-text"><br>
> How do you get the diff out of curiosity?<br>
<br>
</div>It's produced by the `compare.py` script:<br>
<br>
  python test-suite/utils/compare.py -m compile_time unpatched.json patched.json<br>
  python test-suite/utils/compare.py -m size unpatched.json patched.json<br>
<br>
But to run it, you first will need `pip install pandas`, which can take up to 10 minutes :)<br>
<br>
As for the results: even though I'm not super excited with the (mostly) slowdowns, the gain on the real project probably outweigh them, so I think it's the right thing to do.<br>
<br>
One follow-up thought: how hard will it be to run GDCE only on functions? AFAIU, the gain is expected from removing functions that have been inlined and thus no longer need to be compiled. I wonder if we can avoid the slowdowns by looking only for the dead functions when we call GDCE after inlining.<br><br><br>I'll look into it as follow up. Do you have other comments? If not, I'll check this in and port to the new PM</blockquote></div></div></div>