[PATCH] D28312: NewGVN: Fix PR 31501.

[Jack Howarth via Phabricator via llvm-commits]

jwhowarth added a comment.

Confirmed that the proposed patch builds a full 3-stage bootstrap with stage2/stage3 comparison on x86_64-apple-darwin16 for llvm/cfe/clang-tools-extra/polly/openmp/libcxx/compiler-rt when  https://llvm.org/bugs/show_bug.cgi?id=31506 is worked around with...

  Index: lib/Transforms/IPO/PassManagerBuilder.cpp
  ===================================================================
  --- lib/Transforms/IPO/PassManagerBuilder.cpp	(revision 291142)
  +++ lib/Transforms/IPO/PassManagerBuilder.cpp	(working copy)
  @@ -71,7 +71,7 @@ static cl::opt<bool> RunLoadCombine("com
                                       cl::Hidden,
                                       cl::desc("Run the load combining pass"));
   
  -static cl::opt<bool> RunNewGVN("enable-newgvn", cl::init(false), cl::Hidden,
  +static cl::opt<bool> RunNewGVN("enable-newgvn", cl::init(true), cl::Hidden,
                                  cl::desc("Run the NewGVN pass"));
   
   static cl::opt<bool>

instead of passing '-mllvm -enable-newgvn' on CXXFLAGS.


https://reviews.llvm.org/D28312