[PATCH] D32871: [LV] Using VPlan to model the vectorized code and drive its transformation

[Ayal Zaks via Phabricator via llvm-commits]

Ayal updated this revision to Diff 111426.
Ayal edited the summary of this revision.
Ayal added a comment.

Uploading the version updated to top of trunk before committing, including merging with SinkAfter patch https://reviews.llvm.org/D33058 by reordering ingredients before constructing recipes for them.

Also changed InnerLoopVectorizer to be under llvm rather than anonymous namespace, because building with gcc 4.8.5 produced the following warning:

[1/3] Building CXX object lib/Transfor.../LLVMVectorize.dir/LoopVectorize.cpp.o
In file included from llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:50:0:
llvm/lib/Transforms/Vectorize/VPlan.h:194:8: warning: 'llvm::VPTransformState' has a field 'llvm::VPTransformState::ILV' whose type uses the anonymous namespace [enabled by default]
 struct VPTransformState {

  ^

Caught late as building with clang 4.0 did not produce this warning, or any other.


https://reviews.llvm.org/D32871

Files:
  lib/Transforms/Vectorize/CMakeLists.txt
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/AArch64/aarch64-predication.ll
  test/Transforms/LoopVectorize/AArch64/predication_costs.ll
  test/Transforms/LoopVectorize/SystemZ/load-store-scalarization-cost.ll
  test/Transforms/LoopVectorize/first-order-recurrence.ll
  test/Transforms/LoopVectorize/if-pred-non-void.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32871.111426.patch
Type: text/x-patch
Size: 95140 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170816/019d26e7/attachment-0001.bin>