[PATCH] D33685: [InlineCost] Change CallAnalyzer::isGEPFree to use TTI::getUserCost instead of TTI::getGEPCost

Evgeny Astigeevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 31 07:50:51 PDT 2017


eastig added a comment.

We have:

1. We can inform getGEPCost about simplified indices to help it with calculating the cost. But getGEPCost does not take into account the context which GEPs are used in.
2. We have getUserCost which can take the context into account but we cannot inform about simplified indices.

Any ideas how to resolve this?


https://reviews.llvm.org/D33685





More information about the llvm-commits mailing list