[PATCH] D80984: [CostModel] Unify getMemoryOpCost
Sam Parker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 2 01:38:45 PDT 2020
samparker created this revision.
samparker added reviewers: RKSimon, craig.topper, spatel, uweigand.
Herald added subscribers: kbarton, hiraditya, nemanjai.
Herald added a project: LLVM.
samparker added a parent revision: D80924: [CostModel] Use MaybeAlign in getMemoryOpCost.
Herald added a subscriber: wuzish.
Use getMemoryOpCost from the generic implementation of getUserCost and have getInstructionThroughput return the result of that for loads and stores. This also means that the X86 implementation of getUserCost can be removed with the functionality folded into its getMemoryOpCost.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D80984
Files:
llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
llvm/include/llvm/CodeGen/BasicTTIImpl.h
llvm/lib/Analysis/TargetTransformInfo.cpp
llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp
llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
llvm/lib/Target/Hexagon/HexagonTargetTransformInfo.cpp
llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp
llvm/lib/Target/SystemZ/SystemZTargetTransformInfo.cpp
llvm/lib/Target/X86/X86TargetTransformInfo.cpp
llvm/lib/Target/X86/X86TargetTransformInfo.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80984.267812.patch
Type: text/x-patch
Size: 9475 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200602/d13b7bd2/attachment.bin>
More information about the llvm-commits
mailing list