[llvm] r253963 - [TableGen] Use the other version of EnforceVectorEltTypeIs inside the TypeSet version of EnforceVectorEltTypeIs to reduce duplicated code. NFC
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 24 00:20:46 PST 2015
Author: ctopper
Date: Tue Nov 24 02:20:45 2015
New Revision: 253963
URL: http://llvm.org/viewvc/llvm-project?rev=253963&view=rev
Log:
[TableGen] Use the other version of EnforceVectorEltTypeIs inside the TypeSet version of EnforceVectorEltTypeIs to reduce duplicated code. NFC
Modified:
llvm/trunk/utils/TableGen/CodeGenDAGPatterns.cpp
Modified: llvm/trunk/utils/TableGen/CodeGenDAGPatterns.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/TableGen/CodeGenDAGPatterns.cpp?rev=253963&r1=253962&r2=253963&view=diff
==============================================================================
--- llvm/trunk/utils/TableGen/CodeGenDAGPatterns.cpp (original)
+++ llvm/trunk/utils/TableGen/CodeGenDAGPatterns.cpp Tue Nov 24 02:20:45 2015
@@ -491,22 +491,8 @@ bool EEVT::TypeSet::EnforceVectorEltType
MVT::SimpleValueType VT = VTOperand.getConcrete();
- TypeSet InputSet(*this);
+ MadeChange |= EnforceVectorEltTypeIs(VT, TP);
- // Filter out all the types which don't have the right element type.
- for (unsigned i = 0; i != TypeVec.size(); ++i) {
- assert(isVector(TypeVec[i]) && "EnforceVector didn't work");
- if (MVT(TypeVec[i]).getVectorElementType().SimpleTy != VT) {
- TypeVec.erase(TypeVec.begin()+i--);
- MadeChange = true;
- }
- }
-
- if (TypeVec.empty()) { // FIXME: Really want an SMLoc here!
- TP.error("Type inference contradiction found, forcing '" +
- InputSet.getName() + "' to have a vector element");
- return false;
- }
return MadeChange;
}
More information about the llvm-commits
mailing list