[PATCH] D14063: [SLP] Try a bit harder to find reduction PHIs

[Charlie Turner via llvm-commits]

chatur01 created this revision.
chatur01 added reviewers: nadav, jmolloy, mcrosier.
chatur01 added a subscriber: llvm-commits.
chatur01 set the repository for this revision to rL LLVM.
Herald added a subscriber: aemerson.

Currently, when the SLP vectorizer considers whether a phi is part of a reduction, it dismisses phi's whose incoming blocks are not the same as the block containing the phi. For the patterns I'm looking at, extending this rule to allow phis whose incoming block is a containing loop latch allows me to vectorize certain workloads.

There is no significant compile-time impact, and combined with D13949, no performance improvement measured in ARM/AArch64 in any of SPEC2000, SPEC2006 or LNT.

Repository:
  rL LLVM

http://reviews.llvm.org/D14063

Files:
  lib/Transforms/Vectorize/SLPVectorizer.cpp
  test/Transforms/SLPVectorizer/AArch64/horizontal.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D14063.38399.patch
Type: text/x-patch
Size: 5125 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151026/b951bea3/attachment.bin>