avoid constructing an incomplete TTI in InlineCostAnnotationPrinterPass

[vtjnash]

Summary

It is not an essential pass (just printing), but analysis is supposed to come from the FAM, not be constructed here (it is missing the TargetMachine, which would cause it to print the wrong costs).

[vtjnash]

bump?

[vtjnash]

bump? this should be an easy obvious bugfix

[wsmoses]

Since you say it's possible to get the wrong cost, is it possible to create a test case that highlights this?

llvm/lib/Analysis/InlineCost.cpp

 PreservedAnalyses
 InlineCostAnnotationPrinterPass::run(Function &F,
                                      FunctionAnalysisManager &FAM) {
   PrintInstructionComments = true;
   std::function<AssumptionCache &(Function &)> GetAssumptionCache =
       [&](Function &F) -> AssumptionCache & {
     return FAM.getResult<AssumptionAnalysis>(F);
   };
+  TargetTransformInfo &TTI = FAM.getResult<TargetIRAnalysis>(F);
   Module *M = F.getParent();
   ProfileSummaryInfo PSI(*M);
   DataLayout DL(M);
-  TargetTransformInfo TTI(DL);
   // FIXME: Redesign the usage of InlineParams to expand the scope of this pass.
   // In the current implementation, the type of InlineParams doesn't matter as
   // the pass serves only for verification of inliner's decisions.
   // We can add a flag which determines InlineParams for this run. Right now,
   // the default InlineParams are used.
   const InlineParams Params = llvm::getInlineParams();
   for (BasicBlock &BB : F) {
     for (Instruction &I : BB) {