[PATCH] D55894: [X86][SSE] Auto upgrade PADDS/PSUBS intrinsics to SADD_SAT/SSUB_SAT generic intrinsics (llvm)

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon marked an inline comment as done.
RKSimon added inline comments.


================
Comment at: test/Transforms/InstCombine/X86/x86-adds-subs.ll:22
 ; CHECK-LABEL: @sse2_adds_b_constant_overflow(
 ; CHECK-NEXT:    ret <16 x i8> <i8 2, i8 4, i8 6, i8 8, i8 10, i8 12, i8 14, i8 16, i8 18, i8 20, i8 127, i8 24, i8 26, i8 28, i8 30, i8 127>
 ;
----------------
craig.topper wrote:
> Probably should move this out to generic. Feels weird to test upgraded intrinsics.
OK - will do this as a pre-commit if that's OK? What about the UNDEF handling, just raise a bug?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55894