[PATCH] D59014: [TTI] Enable analysis of clib functions in getIntrinsicCosts. NFCI.

[Sam Parker via Phabricator via llvm-commits]

samparker added inline comments.


================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:211
   /// The latter is only interesting for varargs function types.
-  int getCallCost(FunctionType *FTy, int NumArgs = -1) const;
+  int getCallCost(FunctionType *FTy, int NumArgs = -1, User *U = nullptr) const;
 
----------------
Can these be const pointers?


================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:1507
   }
+  int getMemcpyCost(const Instruction *I) {
+    return Impl.getMemcpyCost(I);
----------------
Since you're implementing this, we should have a test.


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

https://reviews.llvm.org/D59014