[llvm] r277685 - GVN-hoist: enable by default

[Bruno Cardoso Lopes via llvm-commits]

Hi Sebastian,

Had to revert it in r277690, since it broke
http://lab.llvm.org:8080/green/job/clang-stage1-configure-RA_build/23335

On Wed, Aug 3, 2016 at 6:59 PM, Sebastian Pop via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: spop
> Date: Wed Aug  3 20:59:42 2016
> New Revision: 277685
>
> URL: http://llvm.org/viewvc/llvm-project?rev=277685&view=rev
> Log:
> GVN-hoist: enable by default
>
> 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.
>
> Differential Revision: https://reviews.llvm.org/D23136
>
> Modified:
>     llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
>
> Modified: llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp?rev=277685&r1=277684&r2=277685&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp (original)
> +++ llvm/trunk/lib/Transforms/IPO/PassManagerBuilder.cpp Wed Aug  3 20:59:42 2016
> @@ -138,8 +138,8 @@ static cl::opt<int> PreInlineThreshold(
>               "(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 GVN hoisting pass (default = on)"));
>
>  PassManagerBuilder::PassManagerBuilder() {
>      OptLevel = 2;
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits



-- 
Bruno Cardoso Lopes
http://www.brunocardoso.cc