[PATCH] D55950: [ConstantFolding] Fold undef for integer intrinsics

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 20 12:46:09 PST 2018


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


================
Comment at: test/Transforms/InstCombine/saturating-add-sub-vector.ll:22
 ; CHECK-LABEL: @sadd_sat_v16i8_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>
 ;
----------------
I think this test could be dropped in favor of `test/Analysis/ConstantFolding/saturating-add-sub.ll`. It should already cover everything, just with not quite as wide vectors...


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55950





More information about the llvm-commits mailing list