[PATCH] D101507: [SimplifyCFG] Ignore llvm.assume when looking for side effects

[Nikita Popov via Phabricator via llvm-commits]

nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LGTM, summary needs an update though.



================
Comment at: llvm/test/Transforms/PhaseOrdering/d83507-knowledge-retention-bug.ll:3
+; RUN: opt -loop-rotate -instcombine -enable-knowledge-retention -S < %s  | FileCheck %s
+; RUN: opt -passes='loop(loop-rotate),instcombine' -enable-knowledge-retention -S < %s  | FileCheck %s
 
----------------
Would it be possible to instead adjust the test to have a non-undef IV?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101507