[PATCH] D13949: [SLP] Treat SelectInsts as reduction values.

[Charlie Turner via llvm-commits]

chatur01 added a comment.

> Do you know how effective is this pattern? Do you know how many times it hits when you compile the llvm test suite?


My testing has been on ARM and AArch64 targets, this patch has no effect on performance yet for the benchmarks I have access to. To vectorize the more general pattern, there are few more changes I need to propose, including searching for reductions in multi-block loops, and trying different reduction widths. I was going to go through them one at a time.

With the other changes, the biggest improvements are in MPEG encoders in third party benchmarks. I haven't yet measured the compile time impact of this particular change, I will get some numbers for this. Thanks for your review!

--Charlie.


Repository:
  rL LLVM

http://reviews.llvm.org/D13949