[PATCH] D113973: [LoopVectorize][CostModel] Choose smaller VFs for in-loop reductions with no loads/stores

Rosie Sumpter via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 16 08:12:17 PST 2021


RosieSumpter updated this revision to Diff 394884.
RosieSumpter added a comment.

- Split the cast instruction SmallPtrSet into 2 separate ones; `CastsToRecurrenceType` and `CastsFromRecurrenceType`
- removed bool `IgnoreCasts` parameter in `collectCastInstrs`
- call `collectCastInsts` for all recurrence descriptors which get saved at the end of `AddReductionVar`
- query `CastsToRecurrenceType` when checking casts used by recurrence descriptors in `getSmallestAndWidestTypes`


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

https://reviews.llvm.org/D113973

Files:
  llvm/include/llvm/Analysis/IVDescriptors.h
  llvm/lib/Analysis/IVDescriptors.cpp
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/smallest-and-widest-types.ll
  llvm/test/Transforms/LoopVectorize/X86/funclet.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D113973.394884.patch
Type: text/x-patch
Size: 12912 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20211216/d370bea9/attachment.bin>


More information about the llvm-commits mailing list