[PATCH] D140853: [ConstraintElim] Move after first instcombine run.

[Nikita Popov via Phabricator via llvm-commits]

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

LG

> There are is a tiny number of regressions, mostly due to CVP eliminating
> a condition that ConstraintElimination would use, but in most cases
> there's a slight improvement or no change.

To clarify, the condition could still be derived from earlier ones, but ConstraintElimination just isn't powerful enough to do that? (E.g. because it requires reasoning about ranges.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140853