[NFCI][CostModel] Unify getCFInstrCost

[samparker]

Summary

Have TTI::getInstructionThroughput call getUserCost for Br, Ret and PHI.

[samparker]

Thanks @dfukalov. Would you mind looking at the AMD changes in D79164 too please?

llvm/lib/Analysis/TargetTransformInfo.cpp

TTI::ReductionKind TTI::matchVectorSplittingReduction(
   return RD->Kind;
 }
 
 int TargetTransformInfo::getInstructionThroughput(const Instruction *I) const {
   TTI::TargetCostKind CostKind = TTI::TCK_RecipThroughput;
 
   switch (I->getOpcode()) {
   case Instruction::GetElementPtr:
-    return getUserCost(I, CostKind);
-
   case Instruction::Ret:
   case Instruction::PHI:
-  case Instruction::Br: {
-    return getCFInstrCost(I->getOpcode(), CostKind);
-  }
+  case Instruction::Br:
   case Instruction::Add:
   case Instruction::FAdd:
   case Instruction::Sub:
   case Instruction::FSub:
   case Instruction::Mul:
   case Instruction::FMul:
   case Instruction::UDiv:
   case Instruction::SDiv:
   case Instruction::FDiv:
   case Instruction::URem:
   case Instruction::SRem:
   case Instruction::FRem:
   case Instruction::Shl:
   case Instruction::LShr:
   case Instruction::AShr:
   case Instruction::And:
   case Instruction::Or:
   case Instruction::Xor:
   case Instruction::FNeg:
-    return getUserCost(I, CostKind);
   case Instruction::Select:
   case Instruction::ICmp:
   case Instruction::FCmp:
   case Instruction::Store:
   case Instruction::Load:
   case Instruction::ZExt:
   case Instruction::SExt:
   case Instruction::FPToUI: