This is an archive of the discontinued LLVM Phabricator instance.

GlobalISel: Assert if MoreElements uses a non-vector type
ClosedPublic

Authored by arsenm on Sep 28 2020, 6:17 AM.

Reviewers

arsenm requested review of this revision.Sep 28 2020, 6:17 AM

This revision is now accepted and ready to land.Sep 28 2020, 10:13 PM

Path

Size

llvm/

lib/

CodeGen/

GlobalISel/

3 lines

Show First 20 Lines • Show All 142 Lines • ▼ Show 20 Lines	if (NewTy.isVector()) {
// Make sure the element count really decreased.		// Make sure the element count really decreased.
if (NewTy.getNumElements() >= OldElts)		if (NewTy.getNumElements() >= OldElts)
return false;		return false;
} else {		} else {
// Make sure the element count really increased.		// Make sure the element count really increased.
if (NewTy.getNumElements() <= OldElts)		if (NewTy.getNumElements() <= OldElts)
return false;		return false;
}		}
}		} else if (Rule.getAction() == MoreElements)
		return false;

// Make sure the element type didn't change.		// Make sure the element type didn't change.
return NewTy.getScalarType() == OldTy.getScalarType();		return NewTy.getScalarType() == OldTy.getScalarType();
}		}
case NarrowScalar:		case NarrowScalar:
case WidenScalar: {		case WidenScalar: {
if (OldTy.isVector()) {		if (OldTy.isVector()) {
// Number of elements should not change.		// Number of elements should not change.
▲ Show 20 Lines • Show All 590 Lines • Show Last 20 Lines