[PATCH] D29196: [X86 Codegen] PACKUS can't be used for unsigned saturation

Zvi Rackover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 29 04:01:35 PST 2017


zvi added a comment.

Other than the comment about the deleted test cases, LGTM



================
Comment at: ../test/CodeGen/X86/avx-trunc.ll:43
 
-define <16 x i8> @usat_trunc_wb_256(<16 x i16> %i) {
-; CHECK-LABEL: usat_trunc_wb_256:
----------------
I suggest you move these deleted functions to a separate file name pr#####.ll with CHECK-NOT's


Repository:
  rL LLVM

https://reviews.llvm.org/D29196





More information about the llvm-commits mailing list