[PATCH] D80550: [NFCI][CostModel] Unify getCmpSelInstrCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 26 06:27:21 PDT 2020


samparker created this revision.
samparker added reviewers: RKSimon, spatel, jsji, uweigand.
Herald added subscribers: kbarton, hiraditya, nemanjai.
Herald added a project: LLVM.

Add cases for icmp, fcmp and select into the switch statement of the generic getUserCost implementation with getInstructionThroughput then calling into it. The BasicTTI and backend implementations have be set to return a default value (1) when a cost other than throughput is being queried.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D80550

Files:
  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/Hexagon/HexagonTargetTransformInfo.cpp
  llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp
  llvm/lib/Target/SystemZ/SystemZTargetTransformInfo.cpp
  llvm/lib/Target/X86/X86TargetTransformInfo.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80550.266177.patch
Type: text/x-patch
Size: 6099 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200526/5aa348e4/attachment.bin>


More information about the llvm-commits mailing list