This is an archive of the discontinued LLVM Phabricator instance.

[NFCI][CostModel] Unify getCFInstrCost
ClosedPublic

Authored by samparker on May 13 2020, 4:41 AM.

Reviewers

Commits

rG7158f285a867: [CostModel] Unify getCFInstrCost

Summary

Have TTI::getInstructionThroughput call getUserCost for Br, Ret and PHI.

This revision is now accepted and ready to land.May 21 2020, 2:39 AM

Thanks @dfukalov. Would you mind looking at the AMD changes in D79164 too please?

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

lib/

Analysis/

7 lines

	Show First 20 Lines • Show All 492 Lines • ▼ Show 20 Lines
	return RD->Kind;			return RD->Kind;
	}			}

	int TargetTransformInfo::getInstructionThroughput(const Instruction *I) const {			int TargetTransformInfo::getInstructionThroughput(const Instruction *I) const {
	TTI::TargetCostKind CostKind = TTI::TCK_RecipThroughput;			TTI::TargetCostKind CostKind = TTI::TCK_RecipThroughput;

	switch (I->getOpcode()) {			switch (I->getOpcode()) {
	case Instruction::GetElementPtr:			case Instruction::GetElementPtr:
	return getUserCost(I, CostKind);

	case Instruction::Ret:			case Instruction::Ret:
	case Instruction::PHI:			case Instruction::PHI:
	case Instruction::Br: {			case Instruction::Br:
	return getCFInstrCost(I->getOpcode(), CostKind);			return getUserCost(I, CostKind);
	}
	case Instruction::Add:			case Instruction::Add:
	case Instruction::FAdd:			case Instruction::FAdd:
	case Instruction::Sub:			case Instruction::Sub:
	case Instruction::FSub:			case Instruction::FSub:
	case Instruction::Mul:			case Instruction::Mul:
	case Instruction::FMul:			case Instruction::FMul:
	case Instruction::UDiv:			case Instruction::UDiv:
	case Instruction::SDiv:			case Instruction::SDiv:
	▲ Show 20 Lines • Show All 227 Lines • Show Last 20 Lines