[PATCH] D148979: [Pipelines] Move SimplifyCFGPass in front of InstCombinePass.

[Nikita Popov via Phabricator via llvm-commits]

nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.

The pattern folds fine in isolation (https://alive2.llvm.org/ce/z/WfqQwW), so I don't understand why a phase ordering change would be needed for this. Most likely this just needs a special case in simplifyWithOpReplaced(), to say that the derefining replacement is safe in this case, based on poison implication.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148979