This is an archive of the discontinued LLVM Phabricator instance.

[GISel: Add support for CSEing SrcOps which are immediates
ClosedPublic

Authored by aditya_nandakumar on Jul 17 2020, 3:06 PM.

Reviewers

Summary

This came up in attempting to CSE a target pseudo opcode.

	Time	Test
	620 ms	windows > Clang.OpenMP::Unknown Unit Message ("")

Couldn't you test this with a G_EXTRACT?

Add G_EXTRACT to CSEConfigFull, and add unit test.

This revision is now accepted and ready to land.Jul 17 2020, 3:44 PM

Path

Size

llvm/

lib/

CodeGen/

GlobalISel/

3 lines

Show First 20 Lines • Show All 64 Lines • ▼ Show 20 Lines	default:
B.addNodeIDRegType(Op.getLLTTy(*getMRI()));		B.addNodeIDRegType(Op.getLLTTy(*getMRI()));
break;		break;
}		}
}		}

void CSEMIRBuilder::profileSrcOp(const SrcOp &Op,		void CSEMIRBuilder::profileSrcOp(const SrcOp &Op,
GISelInstProfileBuilder &B) const {		GISelInstProfileBuilder &B) const {
switch (Op.getSrcOpKind()) {		switch (Op.getSrcOpKind()) {
		case SrcOp::SrcType::Ty_Imm:
		B.addNodeIDImmediate(static_cast<int64_t>(Op.getImm()));
		break;
case SrcOp::SrcType::Ty_Predicate:		case SrcOp::SrcType::Ty_Predicate:
B.addNodeIDImmediate(static_cast<int64_t>(Op.getPredicate()));		B.addNodeIDImmediate(static_cast<int64_t>(Op.getPredicate()));
break;		break;
default:		default:
B.addNodeIDRegType(Op.getReg());		B.addNodeIDRegType(Op.getReg());
break;		break;
}		}
}		}
▲ Show 20 Lines • Show All 175 Lines • Show Last 20 Lines