[PATCH] D101076: [SVE][LoopVectorize] Add support for scalable vectorization of first-order recurrences

[Kerry McLaughlin via Phabricator via llvm-commits]

kmclaughlin updated this revision to Diff 340830.
kmclaughlin marked 8 inline comments as done.
kmclaughlin added a comment.

Addressed review comments from @sdesmalen & @david-arm:

- Replaced uses of `Builder.CreateBinOp` with `Builder.GetSub` in fixFirstOrderRecurrence
- Removed tests which added little value over the exisiting tests in test/Transform/LoopVectorize/first-order-recurrence.ll (PR26734, PR27246, PR30183 & no_sink_after)
- Removed `Ty` from CreateVectorReverse


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

https://reviews.llvm.org/D101076

Files:
  llvm/include/llvm/IR/IRBuilder.h
  llvm/lib/IR/IRBuilder.cpp
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/first-order-recurrence.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101076.340830.patch
Type: text/x-patch
Size: 25181 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210427/20cf5052/attachment-0001.bin>