[PATCH] D30710: [LV] Vectorize GEPs
Matthew Simpson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 21 09:19:19 PDT 2017
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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 29748 bytes
Desc: not available
More information about the llvm-commits