[PATCH] D79941: [NFCI][CostModel] Refactor getIntrinsicInstrCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 15 00:08:36 PDT 2020


samparker added a comment.

I think it's also because there's two cost calculation paths, one that is purely typed based and one that looks at the arguments too. I think because of all the nesting, the arguments can get lost too with the original code. If those cost changes look reasonable to you I'll add them in the follow-up patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79941





More information about the llvm-commits mailing list