[PATCH] D94996: [GVN] do not repeat PRE on failure to split critical edge

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/test/Transforms/GVN/critical-edge-split-failure.ll:2
+; RUN: opt -gvn -S -o - %s | FileCheck %s
+; RUN: opt -passes=gvn -S -o - %s | FileCheck %s
+
----------------
nickdesaulniers wrote:
> This is yet another case where NPM test passes before/after a patch that would be red with LPM. cc @aeubanks 
> 
> I don't know if this test should be under
> 
> llvm/test/Transforms/GVN/PRE/
> rather than
> llvm/test/Transforms/GVN/
> ?
This one is much more surprising, I can't think of anything off the top of my head that would cause the output of running GVN to change between the two pass managers.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94996/new/

https://reviews.llvm.org/D94996