This is an archive of the discontinued LLVM Phabricator instance.

TableGen: Add support for intrinsic patterns with vAny
AbandonedPublic

Authored by tstellar on Jan 31 2017, 11:49 AM.

Reviewers

arsenm
• tstellarAMD

Summary

AMDGPU will start using the vAny type for some of its intrinsics.

Build Status

Buildable 3455
Build 3455: arc lint + arc unit

utils/TableGen/CodeGenDAGPatterns.cpp
163	Typo nowthing

Do we still need this"

In D29342#1406690, @arsenm wrote:

Do we still need this"

I don't think so, but if we do it wouldn't be hard to bring back.

tstellar abandoned this revision.

tstellar edited reviewers, added: • tstellarAMD; removed: tstellar.

Path

Size

utils/

TableGen/

5 lines

Show First 20 Lines • Show All 154 Lines • ▼ Show 20 Lines	bool EEVT::TypeSet::MergeInTypeInfo(const EEVT::TypeSet &InVT, TreePattern &TP){
if (InVT.isCompletelyUnknown() \|\| *this == InVT \|\| TP.hasError())		if (InVT.isCompletelyUnknown() \|\| *this == InVT \|\| TP.hasError())
return false;		return false;

if (isCompletelyUnknown()) {		if (isCompletelyUnknown()) {
*this = InVT;		*this = InVT;
return true;		return true;
}		}

		// Do nowthing if we are merging MVT::Any
		arsenmUnsubmitted Not Done Reply Inline Actions Typo nowthing arsenm: Typo nowthing
		if (InVT.TypeVec[0] == MVT::Any) {
		return false;
		}

assert(!TypeVec.empty() && !InVT.TypeVec.empty() && "No unknowns");		assert(!TypeVec.empty() && !InVT.TypeVec.empty() && "No unknowns");

// Handle the abstract cases, seeing if we can resolve them better.		// Handle the abstract cases, seeing if we can resolve them better.
switch (TypeVec[0]) {		switch (TypeVec[0]) {
default: break;		default: break;
case MVT::iPTR:		case MVT::iPTR:
case MVT::iPTRAny:		case MVT::iPTRAny:
if (InVT.hasIntegerTypes()) {		if (InVT.hasIntegerTypes()) {
▲ Show 20 Lines • Show All 3,658 Lines • Show Last 20 Lines