[CostModel] remove hack for intrinsic cost based on cost type

This hack seems to only have been necessary because of the
constructor bug noted in 33125cf.

Once again, it's hard to prove NFC, but that's the hope...

Author: rotateright

llvm/include/llvm/Analysis/TargetTransformInfoImpl.h

@@ -859,11 +859,6 @@ class TargetTransformInfoImplCRTPBase : public TargetTransformInfoImplBase {
     case Instruction::Call: {
       assert(isa<IntrinsicInst>(U) && "Unexpected non-intrinsic call");
       auto *Intrinsic = cast<IntrinsicInst>(U);
-      // FIXME: We shouldn't have this exception for RecipThroughput.
-      if (CostKind == TTI::TCK_RecipThroughput) {
-        IntrinsicCostAttributes CostAttrs(*Intrinsic);
-        return TargetTTI->getIntrinsicInstrCost(CostAttrs, CostKind);
-      }
       IntrinsicCostAttributes CostAttrs(Intrinsic->getIntrinsicID(), *CB);
       return TargetTTI->getIntrinsicInstrCost(CostAttrs, CostKind);
     }