[PATCH] D106399: [VectorCombine] Widening of partial vector loads

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri updated this revision to Diff 360465.
lebedev.ri marked 4 inline comments as done.
lebedev.ri added a comment.

Thanks for taking a look!
Addressing nits.

In D106399#2893246 <https://reviews.llvm.org/D106399#2893246>, @RKSimon wrote:

> D57059 <https://reviews.llvm.org/D57059> looks to be performing masked loads when the wider range is dereferencable (costs permitting) - see the llvm/test/Transforms/SLPVectorizer/X86/dot-product.ll test changes

Then we'll also need a load unmasking transform.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106399

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/include/llvm/CodeGen/BasicTTIImpl.h
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/Transforms/Vectorize/VectorCombine.cpp
  llvm/test/Transforms/VectorCombine/X86/load-inseltpoison.ll
  llvm/test/Transforms/VectorCombine/X86/load-widening.ll
  llvm/test/Transforms/VectorCombine/X86/load.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106399.360465.patch
Type: text/x-patch
Size: 18700 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210721/f1e53523/attachment.bin>