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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 12 00:50:19 PDT 2022


fhahn added a comment.

Consolidating all cost-query related functions in TTI makes sense to me.



================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:366
+                                       int64_t Scale,
+                                       unsigned AddrSpace) const {
     TargetLoweringBase::AddrMode AM;
----------------
this could be done independently?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117723



More information about the llvm-commits mailing list