[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