[PATCH] D78922: [CostModel] Remove getExtCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 20 23:06:07 PDT 2020


samparker added a comment.

> I believe that adding this would at least cause inaccuracies in the costmodelling over what we have right now, possibly regressions in some cases.

Don't forget that this is just doing what it did before though and hopefully anyone who is really concerned about performance won't be relying on BasicTTI to get it. There's also D78937 <https://reviews.llvm.org/D78937> which may help too.


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

https://reviews.llvm.org/D78922





More information about the llvm-commits mailing list