[llvm] r326708 - TableGen: Resolve all template args simultaneously in ResolveMulticlassDefARgs
Nicolai Haehnle via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 5 07:21:19 PST 2018
Author: nha
Date: Mon Mar 5 07:21:19 2018
New Revision: 326708
URL: http://llvm.org/viewvc/llvm-project?rev=326708&view=rev
Log:
TableGen: Resolve all template args simultaneously in ResolveMulticlassDefARgs
Summary:
Use the new resolver interface more explicitly, and avoid traversing
all the initializers multiple times.
Change-Id: I679e86988b309d19f25e6cca8b0b14ea150198a6
Reviewers: arsenm, craig.topper, tra, MartinO
Subscribers: wdng, llvm-commits
Differential Revision: https://reviews.llvm.org/D43654
Modified:
llvm/trunk/lib/TableGen/TGParser.cpp
Modified: llvm/trunk/lib/TableGen/TGParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/TableGen/TGParser.cpp?rev=326708&r1=326707&r2=326708&view=diff
==============================================================================
--- llvm/trunk/lib/TableGen/TGParser.cpp (original)
+++ llvm/trunk/lib/TableGen/TGParser.cpp Mon Mar 5 07:21:19 2018
@@ -2576,29 +2576,30 @@ bool TGParser::ResolveMulticlassDefArgs(
ArrayRef<Init *> TArgs,
ArrayRef<Init *> TemplateVals,
bool DeleteArgs) {
- // Loop over all of the template arguments, setting them to the specified
- // value or leaving them as the default if necessary.
+ // Set all template arguments to the specified value or leave them as the
+ // default if necessary, then resolve them all simultaneously.
+ MapResolver R(CurRec);
+
for (unsigned i = 0, e = TArgs.size(); i != e; ++i) {
// Check if a value is specified for this temp-arg.
if (i < TemplateVals.size()) {
- // Set it now.
if (SetValue(CurRec, DefmPrefixLoc, TArgs[i], None, TemplateVals[i]))
return true;
-
- // Resolve it next.
- CurRec->resolveReferencesTo(CurRec->getValue(TArgs[i]));
-
- if (DeleteArgs)
- // Now remove it.
- CurRec->removeValue(TArgs[i]);
-
} else if (!CurRec->getValue(TArgs[i])->getValue()->isComplete()) {
return Error(SubClassLoc, "value not specified for template argument #" +
Twine(i) + " (" + TArgs[i]->getAsUnquotedString() +
") of multiclassclass '" + MC.Rec.getNameInitAsString() +
"'");
}
+
+ R.set(TArgs[i], CurRec->getValue(TArgs[i])->getValue());
+
+ if (DeleteArgs)
+ CurRec->removeValue(TArgs[i]);
}
+
+ CurRec->resolveReferences(R);
+
return false;
}
More information about the llvm-commits
mailing list