[PATCH] Vectorize starting from insertelements building a vector

Matt Arsenault Matthew.Arsenault at amd.com
Thu Aug 22 14:13:57 PDT 2013


  I found a problem with this when the insertelements only partially build the vector. The Gather would end up inserting new insertelements at the start of the block before the extract elements producing the scalars, and the function verify would fail.

  I've changed the insert point to be after all the scalars in the place it happened, but I'm not sure this is the right place to fix this since there are other places that use Gather.

Hi nadav,

http://llvm-reviews.chandlerc.com/D1471

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D1471?vs=3659&id=3681#toc

Files:
  lib/Transforms/Vectorize/SLPVectorizer.cpp
  test/Transforms/SLPVectorizer/X86/insert-element-build-vector.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1471.4.patch
Type: text/x-patch
Size: 10245 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130822/68be4c69/attachment.bin>


More information about the llvm-commits mailing list