This is an archive of the discontinued LLVM Phabricator instance.

TTI: Fix not using overload of getIntrinsicInstrCost
ClosedPublic

Authored by arsenm on Mar 2 2016, 7:10 PM.

Reviewers

hfinkel

Summary

This was always calling the generic version, so the target
custom implementation was never called.

arsenm retitled this revision from to TTI: Fix not using overload of getIntrinsicInstrCost.

arsenm updated this object.

arsenm added a reviewer: hfinkel.

arsenm added a subscriber: llvm-commits.

LGTM

This revision is now accepted and ready to land.Mar 2 2016, 7:51 PM

r262585

Path

Size

include/

llvm/

CodeGen/

2 lines

Show First 20 Lines • Show All 580 Lines • ▼ Show 20 Lines	public:
/// Get intrinsic cost based on arguments		/// Get intrinsic cost based on arguments
unsigned getIntrinsicInstrCost(Intrinsic::ID IID, Type *RetTy,		unsigned getIntrinsicInstrCost(Intrinsic::ID IID, Type *RetTy,
ArrayRef<Value *> Args) {		ArrayRef<Value *> Args) {
switch (IID) {		switch (IID) {
default: {		default: {
SmallVector<Type *, 4> Types;		SmallVector<Type *, 4> Types;
for (Value *Op : Args)		for (Value *Op : Args)
Types.push_back(Op->getType());		Types.push_back(Op->getType());
return getIntrinsicInstrCost(IID, RetTy, Types);		return static_cast<T *>(this)->getIntrinsicInstrCost(IID, RetTy, Types);
}		}
case Intrinsic::masked_scatter: {		case Intrinsic::masked_scatter: {
Value *Mask = Args[3];		Value *Mask = Args[3];
bool VarMask = !isa<Constant>(Mask);		bool VarMask = !isa<Constant>(Mask);
unsigned Alignment = cast<ConstantInt>(Args[2])->getZExtValue();		unsigned Alignment = cast<ConstantInt>(Args[2])->getZExtValue();
return		return
static_cast<T *>(this)->getGatherScatterOpCost(Instruction::Store,		static_cast<T *>(this)->getGatherScatterOpCost(Instruction::Store,
Args[0]->getType(),		Args[0]->getType(),
▲ Show 20 Lines • Show All 259 Lines • Show Last 20 Lines