[PATCH] D63950: [Codegen][SelectionDAG] X u% C == 0 fold: non-splat vectors, accept urem-by-one lines too

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 28 12:28:07 PDT 2019


lebedev.ri added a comment.

Actually, either i'm missing something damningly obvious, or Hacker's Delight is misleading here.
Even without special-casing, we'd still get `Q=-1` from the algorithm, and we don't care about `P`/`K`.
So there is no real need to special-case `D=1` case.
Let me redo all this..


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63950





More information about the llvm-commits mailing list