[PATCH] D88475: [InstCombine] visitTrunc - trunc (lshr (sext A), C) --> (ashr A, C) non-uniform support

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

In D88475#2300260 <https://reviews.llvm.org/D88475#2300260>, @lebedev.ri wrote:

> Hm, why do we need that clamping here?
> If it's for `undef`s, see `Constant::replaceUndefsWith()`.

Isn't it matching the existing std::min calls ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88475