[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