[PATCH] Include legalization cost when computing scalarization cost

Arnold Schwaighofer aschwaighofer at apple.com
Thu Mar 6 18:56:34 PST 2014


On Mar 6, 2014, at 6:48 PM, Arnold Schwaighofer <aschwaighofer at apple.com> wrote:
> Why do we want to do it at this (higher) level? I think, getVectorInstrCost() should have included the type legalization cost of the scalar type.
> 
> unsigned BasicTTI::getVectorInstrCost(unsigned Opcode, Type *Val,
>                                      unsigned Index) const {
>    return getTLI().getTypeLegalizationCost(Ty->getScalarType()).second;
This should have been:
  return getTLI().getTypeLegalizationCost(Ty->getScalarType()).first;



More information about the llvm-commits mailing list