[PATCH] D23136: GVN-hoist: enable by default

Sebastian Pop via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 3 14:12:24 PDT 2016


sebpop created this revision.
sebpop added a reviewer: dberlin.
sebpop added subscribers: llvm-commits, hiraditya.
Herald added a subscriber: mehdi_amini.

As we addressed all compilation time problems with GVN-hoist
https://llvm.org/bugs/show_bug.cgi?id=28670 
this patch turns GVN-hoist back by default.


https://reviews.llvm.org/D23136

Files:
  lib/Transforms/IPO/PassManagerBuilder.cpp

Index: lib/Transforms/IPO/PassManagerBuilder.cpp
===================================================================
--- lib/Transforms/IPO/PassManagerBuilder.cpp
+++ lib/Transforms/IPO/PassManagerBuilder.cpp
@@ -138,8 +138,8 @@
              "(default = 75)"));
 
 static cl::opt<bool> EnableGVNHoist(
-    "enable-gvn-hoist", cl::init(false), cl::Hidden,
-    cl::desc("Enable the experimental GVN Hoisting pass"));
+    "enable-gvn-hoist", cl::init(true), cl::Hidden,
+    cl::desc("Enable the experimental GVN Hoisting pass (default = on)"));
 
 PassManagerBuilder::PassManagerBuilder() {
     OptLevel = 2;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23136.66714.patch
Type: text/x-patch
Size: 612 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160803/9b2c6646/attachment.bin>


More information about the llvm-commits mailing list