[PATCH] D70051: [Codegen] TargetLowering::prepareUREMEqFold(): `x u% C1 ==/!= C2` with tautological C1 u<= C2 (PR35479)

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

To spell it out: yes, we don't need to special-case **all** tautological cases
like that, only those where `C2 u> (-1 %u C1)`.
For now i didn't see any benefit in that - do we know that there's vector code out there
which intentionally contains such tautological lanes, and that would be benefitted
if we only special-case `C2 u> (-1 %u C1)` ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70051