[PATCH] D106653: [LoopVectorize][AArch64] Enable ordered reductions by default for AArch64

[David Sherwood via Phabricator via llvm-commits]

david-arm created this revision.
david-arm added reviewers: sdesmalen, kmclaughlin, c-rhodes, spatel.
Herald added subscribers: danielkiss, hiraditya, kristof.beyls.
david-arm requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

I have changed the EnableStrictReductions option in LoopVectorize.cpp to
be true by default and added a new TTI interface called
enableStrictReductions() that controls whether or not ordered reductions
should be enabled for a given target. By default this returns false,
whereas for AArch64 it returns true and we rely upon the cost model to
make sensible vectorisation choices.

I have added a new RUN line to show that we use ordered reductions by
default for AArch64:

  Transforms/LoopVectorize/AArch64/strict-fadd.ll


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D106653

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/lib/Analysis/TargetTransformInfo.cpp
  llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h
  llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
  llvm/test/Transforms/LoopVectorize/AArch64/strict-fadd.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D106653.361161.patch
Type: text/x-patch
Size: 5813 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210723/a6f28fe7/attachment.bin>