<div dir="ltr">LLVM has a wonderful policy regarding broken commits: we revert to green. We ask that a test case be available within a reasonable time frame (preferably before, but some exceptions can be made), but otherwise we revert the offending patch, even if it contains nice features that people want, and keep the tree green. This is an awesome policy.<div><br></div><div>I would like to suggest we adopt and follow the same policy for compile time regressions that are large, and especially for ones that are super-linear. As an example from the previous thread:<br><div><br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 14, 2016 at 12:14 PM Bruno Cardoso Lopes via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> There is a possibility that r259673 could play a role here.<br>
><br>
> For the buildSchedGraph() method, there is the -dag-maps-huge-region that<br>
> has the default value of 1000. When I commited the patch, I was expecting<br>
> people to lower this value as needed and also suggested this, but this has<br>
> not happened. 1000 is very high, basically "unlimited".<br>
><br>
> It would be interesting to see what results you get with e.g. -mllvm<br>
> -dag-maps-huge-region=50. Of course, since this is a trade-off between<br>
> compile time and scheduler freedom, some care should be taken before<br>
> lowering this in trunk.<br>
<br>
Indeed we hit this internally, filed a PR:<br>
<a href="https://llvm.org/bugs/show_bug.cgi?id=26940" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=26940</a></blockquote><div><br></div><div>I think we should have rolled back r259673 as soon as the test case was available.</div><div><br></div><div>Thoughts?</div></div></div></div></div>