[PATCH] D112725: [SVE][LoopVectorize] Extract the last lane from a uniform store

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 29 09:19:07 PDT 2021


kmclaughlin updated this revision to Diff 383385.
kmclaughlin marked 3 inline comments as done.
kmclaughlin added a comment.

- Removed redundant `Legal->isUniformMemOp(I)` check from setCostBasedWideningDecision
- Added a comment to VPReplicateRecipe::execute
- Removed the `State.VF.isScalable()` check from VPReplicateRecipe::execute & updated the tests affected by this change. Also added a test of uniform stores for fixed-width.


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

https://reviews.llvm.org/D112725

Files:
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/sve-uniform-store.ll
  llvm/test/Transforms/LoopVectorize/X86/cost-model-assert.ll
  llvm/test/Transforms/LoopVectorize/X86/illegal-parallel-loop-uniform-write.ll
  llvm/test/Transforms/LoopVectorize/X86/uniform_mem_op.ll
  llvm/test/Transforms/LoopVectorize/pr44488-predication.ll
  llvm/test/Transforms/LoopVectorize/pr47343-expander-lcssa-after-cfg-update.ll
  llvm/test/Transforms/LoopVectorize/skeleton-lcssa-crash.ll
  llvm/test/Transforms/LoopVectorize/uniform-store.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D112725.383385.patch
Type: text/x-patch
Size: 30491 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211029/c293abfd/attachment.bin>


More information about the llvm-commits mailing list