[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
Tue May 30 12:54:48 PDT 2017


eastig added a comment.

It's NFC. 'getUserCost' does not do any analysis at the moment, just calls getGEPCost.



================
Comment at: lib/Analysis/InlineCost.cpp:352
      else
        Indices.push_back(*I);
+  return TargetTransformInfo::TCC_Free == TTI.getUserCost(&GEP);
----------------
efriedma wrote:
> Dead loop?
Yep. 


https://reviews.llvm.org/D33685





More information about the llvm-commits mailing list