[PATCH] D108049: [InstCombine] Extend canonicalizeClampLike to handle truncated inputs

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 15 09:25:56 PDT 2021


lebedev.ri added a comment.

I notice that all the changed tests perform signed clamping,
and likewise, you `sext`. Is `sext` always the right choice?
Please add at least one test with unsigned clamp,
and an alive proof.


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

https://reviews.llvm.org/D108049



More information about the llvm-commits mailing list