[PATCH] D79418: [LSR][ARM] Add new TTI hook to mark some LSR chains as profitable

Pierre van Houtryve via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 7 06:45:50 PDT 2020


Pierre-vh updated this revision to Diff 262637.
Pierre-vh retitled this revision from "[LSR][ARM] Add new TTI hook to discard unwanted LSRUses" to "[LSR][ARM] Add new TTI hook to mark some LSR chains as profitable".
Pierre-vh added a comment.

Changing the implementation of the patch.

- TTI hook renamed to `isProfitableLSRChainElement`
  - It now returns true for the VCTP
- Removed `FilterOutUndesirableUses`
- Now `isProfitableLSRChainElement` is called in LSR's `isProfitableChain` function. If it returns true for one of the chain's UserInst, the chain will be considered profitable and will not be optimized by LSR.


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

https://reviews.llvm.org/D79418

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/Target/ARM/ARMTargetTransformInfo.cpp
  llvm/lib/Target/ARM/ARMTargetTransformInfo.h
  llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp
  llvm/test/Transforms/LoopStrengthReduce/ARM/vctp-chains.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79418.262637.patch
Type: text/x-patch
Size: 23655 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200507/5ad6a7e5/attachment.bin>


More information about the llvm-commits mailing list