[PATCH] D87145: [SelectionDAG] Remove an early-out from computeKnownBits for smin/smax

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 7 18:22:53 PDT 2020


craig.topper added inline comments.


================
Comment at: llvm/test/CodeGen/X86/avx512-trunc.ll:1020
 ; SKX-NEXT:    vzeroupper
 ; SKX-NEXT:    retq
   %x1 = icmp sgt <16 x i16> %i, <i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0, i16 0>
----------------
RKSimon wrote:
> Is this testing what it means to? I can't remember offhand what the test is for - @craig.topper any ideas?
Not sure either.  InstCombine simplifies it to a store of all ones.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87145



More information about the llvm-commits mailing list