<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 22, 2014 at 6:14 PM, Michael Spencer <span dir="ltr"><<a href="mailto:bigcheesegs@gmail.com" target="_blank">bigcheesegs@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":6t6" class="a3s" style="overflow:hidden">I ran test-suite with this pass and got some interesting results. Most tests have no changes, but 4 tests showed significant change.<br>

<br>
+ is faster, - is slower.<br>
<br>
| nts.MultiSource/Benchmarks/MiBench/security-sha/security-sha | 44% |<br>
| nts.MultiSource/Benchmarks/Prolangs-C/agrep/agrep | 12% |<br>
| nts.MultiSource/Benchmarks/SciMark2-C/scimark2 | -21% |<br>
| nts.MultiSource/Benchmarks/tramp3d-v4/tramp3d-v4 | -39% | (this drops to -20% if we fix the load splicing cost model in DAGCombine)<br>
<br>
Full data: <a href="https://docs.google.com/spreadsheets/d/13_4ZUBQQYXhexrMzVJ5VICNuLzUhovU8lOl0Rr6wG10/edit?usp=sharing" target="_blank">https://docs.google.com/spreadsheets/d/13_4ZUBQQYXhexrMzVJ5VICNuLzUhovU8lOl0Rr6wG10/edit?usp=sharing</a><br>

<br>
I'd like to commit this disabled by default while the regressions are fixed in tree.</div></blockquote></div><br>For full context, Michael chatted with me on IRC about some aspects of the regressions, and they sound likely to be surprising interactions between this pass and GVN, the inliner, or SROA. Due to the nature of pass-interaction bugs, it seems likely that the fixes won't change this pass substantially, and may require reproducing these issues for other devs to get help. That's why I suggested getting this into the tree, but disabling it by default, and then iterating on the remaining regressions in-tree.</div>
</div>