[PATCH] D106745: [DAG] DAGCombiner::useDivRem - recognise sub(X,mul(div(X,Y),Y)) as a rem(X,Y)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

I think this is mainly a bug in `X86TTIImpl::hasDivRemOp()`.
Why do we decide here in backend that we are allowed to produce divrem, but the pass is told that is is not allowed to?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106745