[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