[PATCH] D91398: [LoopVectorizer] Lower uniform loads as a single load (instead of relying on CSE)

[Philip Reames via Phabricator via llvm-commits]

reames updated this revision to Diff 305029.
reames edited the summary of this revision.
reames added a comment.

Rebase w/over landed tests.

As part of that, include a slight broadening of scope.  For some reason, the code wasn't considering any operand which wasn't an instruction to be uniform.  Since I happened to write my test with memory addresses as arguments, this fell out.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D91398/new/

https://reviews.llvm.org/D91398

Files:
  llvm/include/llvm/Transforms/Vectorize/LoopVectorizationLegality.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/X86/cost-model-assert.ll
  llvm/test/Transforms/LoopVectorize/X86/uniform_mem_op.ll
  llvm/test/Transforms/LoopVectorize/multiple-strides-vectorization.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91398.305029.patch
Type: text/x-patch
Size: 32228 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201113/17a443ac/attachment.bin>