[PATCH] D98152: [InstCombine] Canonicalize SPF to min/max intrinsics

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

Thanks for sharing this! I added a couple of 'not' op enhancements to instcombine, so we should be slightly closer now.



================
Comment at: llvm/test/Transforms/PhaseOrdering/X86/vector-reductions-expanded.ll:153
+; CHECK-NEXT:    [[TMP6:%.*]] = tail call i32 @llvm.smin.i32(i32 [[TMP5]], i32 [[TMP4]])
+; CHECK-NEXT:    ret i32 [[TMP6]]
 ;
----------------
nikic wrote:
> No longer vectorized?
This requires rework in SLP. It assumes that min/max patterns are only in cmp+sel form. I made it work for FP (maxnum/minnum) already, so I'll take a look.


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

https://reviews.llvm.org/D98152