[PATCH] D75908: [SCEV] isHighCostExpansionHelper(): use correct TTI hooks

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 12 08:41:03 PDT 2020


reames added a comment.

In D75908#1916313 <https://reviews.llvm.org/D75908#1916313>, @samparker wrote:

> Agreed that it's broken, but it looks like SCEV expander is the only place that this API is/was used. Can we now remove getOperationCost?


If the code is dead, sure.  That's a great way to fix broken code.  :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75908





More information about the llvm-commits mailing list