[PATCH] D59710: [SLP] remove lower limit for forming reduction patterns

[Sanjay Patel via Phabricator via llvm-commits]

spatel updated this revision to Diff 231625.
spatel added a comment.
This revision is now accepted and ready to land.

Patch updated:
Try a different limitation on the 2-way reduction patterns that we consider as candidates. Here I've limited it to compare (boolean type) reductions to avoid regressions on math reductions. This catches the motivating cmp cases from PR39665 and doesn't seem to interfere with any existing cmp vectorization regression tests.


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

https://reviews.llvm.org/D59710

Files:
  llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
  llvm/test/Transforms/SLPVectorizer/X86/fabs-cost-softfp.ll
  llvm/test/Transforms/SLPVectorizer/X86/horizontal-list.ll
  llvm/test/Transforms/SLPVectorizer/X86/reduction2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59710.231625.patch
Type: text/x-patch
Size: 22272 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191201/1d8d99d3/attachment.bin>