[PATCH] D150093: [KnownBits] Add implementation for `KnownBits::sdiv`

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 8 04:02:35 PDT 2023


nikic added a comment.

FWIW, if we know the sign bits of sdiv, we should really always be converting it to udiv. We actually already do this in CVP, and should also implement the same in InstCombine. It currently only handles the non-negative case, but the others can be implemented by adding the necessary negations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150093



More information about the llvm-commits mailing list