<span>I'm fine with this for 4.0. the testcase it was slow on is a testcase things like licm take hundreds of seconds on. In fact, gvn hoist is now the fastest pass on it that does optimization</span><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 3, 2016, 2:12 PM Sebastian Pop <<a href="mailto:sebpop@gmail.com">sebpop@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">sebpop created this revision.<br>
sebpop added a reviewer: dberlin.<br>
sebpop added subscribers: llvm-commits, hiraditya.<br>
Herald added a subscriber: mehdi_amini.<br>
<br>
As we addressed all compilation time problems with GVN-hoist<br>
<a href="https://llvm.org/bugs/show_bug.cgi?id=28670" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=28670</a><br>
this patch turns GVN-hoist back by default.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D23136" rel="noreferrer" target="_blank">https://reviews.llvm.org/D23136</a><br>
<br>
Files:<br>
  lib/Transforms/IPO/PassManagerBuilder.cpp<br>
<br>
Index: lib/Transforms/IPO/PassManagerBuilder.cpp<br>
===================================================================<br>
--- lib/Transforms/IPO/PassManagerBuilder.cpp<br>
+++ lib/Transforms/IPO/PassManagerBuilder.cpp<br>
@@ -138,8 +138,8 @@<br>
              "(default = 75)"));<br>
<br>
 static cl::opt<bool> EnableGVNHoist(<br>
-    "enable-gvn-hoist", cl::init(false), cl::Hidden,<br>
-    cl::desc("Enable the experimental GVN Hoisting pass"));<br>
+    "enable-gvn-hoist", cl::init(true), cl::Hidden,<br>
+    cl::desc("Enable the experimental GVN Hoisting pass (default = on)"));<br>
<br>
 PassManagerBuilder::PassManagerBuilder() {<br>
     OptLevel = 2;<br>
<br>
<br>
</blockquote></div>