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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 30 10:51:57 PDT 2017


efriedma added a comment.

Is there any functional change here?  (If there is, needs a test; if not, please add a note to the commit message.)



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


https://reviews.llvm.org/D33685





More information about the llvm-commits mailing list