From 1e19e4613913b02761ac736a01f22a3028679f45 Mon Sep 17 00:00:00 2001 From: Nadav Rotem Date: Thu, 25 Oct 2012 18:17:48 +0000 Subject: [PATCH] Minor cleanups. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166706 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Target/TargetTransformImpl.cpp | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/lib/Target/TargetTransformImpl.cpp b/lib/Target/TargetTransformImpl.cpp index 382eecb7665..c8bd80ea256 100644 --- a/lib/Target/TargetTransformImpl.cpp +++ b/lib/Target/TargetTransformImpl.cpp @@ -123,7 +123,7 @@ int InstructionOpcodeToISD(unsigned Opcode) { return OpToISDTbl[Opcode - 1]; } -std::pair +std::pair VectorTargetTransformImpl::getTypeLegalizationCost(LLVMContext &C, EVT Ty) const { unsigned Cost = 1; @@ -150,17 +150,17 @@ VectorTargetTransformImpl::getInstrCost(unsigned Opcode, Type *Ty1, // Check if any of the operands are vector operands. int ISD = InstructionOpcodeToISD(Opcode); - // Selects on vectors are actually vector selects. - if (ISD == ISD::SELECT) { - assert(Ty2 && "Ty2 must hold the select type"); - if (Ty2->isVectorTy()) - ISD = ISD::VSELECT; - } - // If we don't have any information about this instruction assume it costs 1. if (ISD == 0) return 1; + // Selects on vectors are actually vector selects. + if (ISD == ISD::SELECT) { + assert(Ty2 && "Ty2 must hold the condition type"); + if (Ty2->isVectorTy()) + ISD = ISD::VSELECT; + } + assert(Ty1 && "We need to have at least one type"); // From this stage we look at the legalized type. @@ -174,7 +174,7 @@ VectorTargetTransformImpl::getInstrCost(unsigned Opcode, Type *Ty1, } unsigned NumElem = - (LT.second.isVector() ? LT.second.getVectorNumElements() : 1); + (LT.second.isVector() ? LT.second.getVectorNumElements() : 1); // We will probably scalarize this instruction. Assume that the cost is the // number of the vector elements.