[PATCH] D76124: [TTI] Remove getOperationCost

[Sam Parker via Phabricator via llvm-commits]

samparker created this revision.
samparker added reviewers: RKSimon, jnspaulsson, Carrot, craig.topper, spatel, fhahn, dmgreen, greened.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.

This API call has been used recently with, a very valid, expectation that it would do something useful but it doesn't actually query any backend information. So, remove this method and merge its functionality into getUserCost. As well as that, also use getCastInstrCost to get a proper cost from the backend for the concerned instructions which compensates for the removal of the BasicTTI layer. The next step would be to use other useful API calls in getUserCost too.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D76124

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/test/Analysis/CostModel/SystemZ/ext-of-icmp-cost.ll
  llvm/test/Analysis/CostModel/X86/costmodel.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D76124.250167.patch
Type: text/x-patch
Size: 14362 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200313/b42bebfd/attachment.bin>