[PATCH] D56636: [CodeGen][X86] Expand vector USUBSAT to UMAX+SUB

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 14 05:59:16 PST 2019


nikic marked an inline comment as done.
nikic added a comment.

In D56636#1356032 <https://reviews.llvm.org/D56636#1356032>, @RKSimon wrote:

> X86 vXi64 costs?


I've added vXi64 costs only for AVX512, as pmaxuq wasn't available previously. Should I also add costs based on the broadcast+2*xor+cmpgt+blend+sub sequence we get if there is no native umax?


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

https://reviews.llvm.org/D56636





More information about the llvm-commits mailing list