[PATCH] D78922: [CostModel] Remove getExtCost

[Sam Parker via Phabricator via llvm-commits]

samparker updated this revision to Diff 261824.
samparker retitled this revision from "[NFCI][CostModel] Remove getExtCost" to "[CostModel] Remove getExtCost".
samparker added reviewers: uweigand, dmgreen.
samparker added a comment.
Herald added subscribers: kbarton, nemanjai.

After turning on asserts, two inlining tests were failing for AArch64 and PowerPC. I've now updated their implementations of getCastInstrCost to pass the Instruction to the default implementation. I've also done the same for the ARM backend.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78922/new/

https://reviews.llvm.org/D78922

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/AArch64/AArch64TargetTransformInfo.cpp
  llvm/lib/Target/ARM/ARMTargetTransformInfo.cpp
  llvm/lib/Target/PowerPC/PPCTargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/ARM/cast.ll
  llvm/test/Transforms/SLPVectorizer/AArch64/horizontal.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78922.261824.patch
Type: text/x-patch
Size: 15777 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200504/2ecb0146/attachment.bin>