[PATCH] D28312: NewGVN: Fix PR 31501.

[Jack Howarth via Phabricator via llvm-commits]

jwhowarth added a comment.

In https://reviews.llvm.org/D28312#638623, @davide wrote:

> In https://reviews.llvm.org/D28312#638616, @jwhowarth wrote:
>
> > Perhaps also flip the default on the RunNewGVN enable-newgvn opt to true to see if any remaining latent regressions can be smoked out on less common targets?
>
>
> We're nowhere near there yet, unfortunately.


Enabling NewGVN here produces only 11 regressions in the llvm test suite on x86_64-apple-darwin16...

Failing Tests (11):

    LLVM :: BugPoint/compile-custom.ll
    LLVM :: BugPoint/crash-narrowfunctiontest.ll
    LLVM :: BugPoint/invalid-debuginfo.ll
    LLVM :: BugPoint/metadata.ll
    LLVM :: BugPoint/named-md.ll
    LLVM :: BugPoint/remove_arguments_test.ll
    LLVM :: BugPoint/replace-funcs-with-null.ll
    LLVM :: Transforms/Coroutines/ArgAddr.ll
    LLVM :: Transforms/Coroutines/ex5.ll
    LLVM :: Transforms/GVN/no_speculative_loads_with_asan.ll
    LLVM :: Transforms/NewGVN/no_speculative_loads_with_asan.ll
   
  Expected Passes    : 14131
  Expected Failures  : 80
  Unsupported Tests  : 4996
  Unexpected Failures: 11


Repository:
  rL LLVM

https://reviews.llvm.org/D28312