[PATCH] D94693: Improve KnownBits analyses for SMIN/SMAX DAG nodes.

Ayman Musa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 08:30:25 PST 2021


aymanmus added a comment.

@foad, @RKSimon, Thanks for directing me to the other patch.
I'll just close this and track D87145 <https://reviews.llvm.org/D87145>.



================
Comment at: llvm/test/CodeGen/X86/avx512-trunc.ll:1018
+; SKX-NEXT:    vpcmpeqd %ymm0, %ymm0, %ymm0
 ; SKX-NEXT:    vpmovwb %ymm0, (%rdi)
 ; SKX-NEXT:    vzeroupper
----------------
RKSimon wrote:
> Same problem as D87145 - this isn't testing what its supposed to be testing.....
I agree, the test doesn't make sense.


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

https://reviews.llvm.org/D94693



More information about the llvm-commits mailing list