[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