[PATCH] D117723: [NFCI] Move cost estimation from TargetLowering to TargetTransformInfo.

[Daniil Fukalov via Phabricator via llvm-commits]

dfukalov added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:366
+                                       int64_t Scale,
+                                       unsigned AddrSpace) const {
     TargetLoweringBase::AddrMode AM;
----------------
fhahn wrote:
> this could be done independently?
Actually I added `const` since the removed `TargetLoweringBase::getScalingFactorCost()` was const.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117723