[PATCH] D30710: [LV] Vectorize GEPs

[Matthew Simpson via Phabricator via llvm-commits]

mssimpso updated this revision to Diff 92494.
mssimpso added a comment.

Incorporated feedback from Elena and Michael. Thanks!

- I separated out the loop-invariant case to make the code more explicit. But I'm still vectorizing this case rather than scalarizing it as suggested by Elena. I don't think we need to revisit the scalarization decision here.
- I added an assert for the loop-varying case to ensure we actually create a vector of pointers.
- I rewrote the comments to hopefully add some more clarity.


https://reviews.llvm.org/D30710

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/X86/consecutive-ptr-uniforms.ll
  test/Transforms/LoopVectorize/X86/scatter_crash.ll
  test/Transforms/LoopVectorize/vector-geps.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30710.92494.patch
Type: text/x-patch
Size: 29748 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170321/96ad108e/attachment.bin>