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

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D59710#1764582 <https://reviews.llvm.org/D59710#1764582>, @spatel wrote:

> 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.


It looks mostly like a hack. And I assume in some cases it still may lead to problems with performance. Better to try to fix the cost model for x86, I think.


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

https://reviews.llvm.org/D59710