[PATCH] D77050: [TTI] Remove getCallCost

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added a comment.

This is a tricky one, because usually it is quite simpel: if code is not used, it should be deleted.  However, estimating costs of e.g. libc functions is such a glaring omission, something we definitely should be doing, but we aren't very unfortunately. I had to switch tasks before I could finish D59129 <https://reviews.llvm.org/D59129>, but that is just one example how we could estimate costs.... So yeah, I don't know, if this is really bothering you, we can get rid of rid, but perhaps keeping this infrastructure isn't that bad?
Perhaps this is also a good motivation to pick up D59129 <https://reviews.llvm.org/D59129> again on the side...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77050