Revert "Improve memory ownership/management in TableGen by unique_ptrifying TreePattern's Tree member."
This reverts commit r222183.
Broke on the MSVC buildbots due to MSVC not producing default move
operations - I'd fix it immediately but just broke my build system a
bit, so backing out until I have a chance to get everything going again.
llvm-svn: 222187
diff --git a/llvm/utils/TableGen/CodeGenDAGPatterns.cpp b/llvm/utils/TableGen/CodeGenDAGPatterns.cpp
index 26f32d2..a750aa9 100644
--- a/llvm/utils/TableGen/CodeGenDAGPatterns.cpp
+++ b/llvm/utils/TableGen/CodeGenDAGPatterns.cpp
@@ -1902,10 +1902,10 @@
Trees.push_back(ParseTreePattern(Pat, ""));
}
-TreePattern::TreePattern(Record *TheRec, std::unique_ptr<TreePatternNode> Pat, bool isInput,
+TreePattern::TreePattern(Record *TheRec, TreePatternNode *Pat, bool isInput,
CodeGenDAGPatterns &cdp) : TheRecord(TheRec), CDP(cdp),
isInputPattern(isInput), HasError(false) {
- Trees.push_back(std::move(Pat));
+ Trees.push_back(Pat);
}
void TreePattern::error(const Twine &Msg) {
@@ -1918,7 +1918,7 @@
void TreePattern::ComputeNamedNodes() {
for (unsigned i = 0, e = Trees.size(); i != e; ++i)
- ComputeNamedNodes(Trees[i].get());
+ ComputeNamedNodes(Trees[i]);
}
void TreePattern::ComputeNamedNodes(TreePatternNode *N) {
@@ -1929,8 +1929,8 @@
ComputeNamedNodes(N->getChild(i));
}
-std::unique_ptr<TreePatternNode>
-TreePattern::ParseTreePattern(Init *TheInit, StringRef OpName) {
+
+TreePatternNode *TreePattern::ParseTreePattern(Init *TheInit, StringRef OpName){
if (DefInit *DI = dyn_cast<DefInit>(TheInit)) {
Record *R = DI->getDef();
@@ -1944,7 +1944,7 @@
OpName);
// Input argument?
- auto Res = llvm::make_unique<TreePatternNode>(DI, 1);
+ TreePatternNode *Res = new TreePatternNode(DI, 1);
if (R->getName() == "node" && !OpName.empty()) {
if (OpName.empty())
error("'node' argument requires a name to match with operand list");
@@ -1959,7 +1959,7 @@
if (TheInit == UnsetInit::get()) {
if (OpName.empty())
error("'?' argument requires a name to match with operand list");
- auto Res = llvm::make_unique<TreePatternNode>(TheInit, 1);
+ TreePatternNode *Res = new TreePatternNode(TheInit, 1);
Args.push_back(OpName);
Res->setName(OpName);
return Res;
@@ -1968,7 +1968,7 @@
if (IntInit *II = dyn_cast<IntInit>(TheInit)) {
if (!OpName.empty())
error("Constant int argument should not have a name!");
- return llvm::make_unique<TreePatternNode>(II, 1);
+ return new TreePatternNode(II, 1);
}
if (BitsInit *BI = dyn_cast<BitsInit>(TheInit)) {
@@ -1994,7 +1994,7 @@
if (Dag->getNumArgs() != 1)
error("Type cast only takes one operand!");
- auto New = ParseTreePattern(Dag->getArg(0), Dag->getArgName(0));
+ TreePatternNode *New = ParseTreePattern(Dag->getArg(0), Dag->getArgName(0));
// Apply the type cast.
assert(New->getNumTypes() == 1 && "FIXME: Unhandled");
@@ -2044,8 +2044,7 @@
// Parse all the operands.
for (unsigned i = 0, e = Dag->getNumArgs(); i != e; ++i)
- Children.push_back(
- ParseTreePattern(Dag->getArg(i), Dag->getArgName(i)).release());
+ Children.push_back(ParseTreePattern(Dag->getArg(i), Dag->getArgName(i)));
// If the operator is an intrinsic, then this is just syntactic sugar for for
// (intrinsic_* <number>, ..children..). Pick the right intrinsic node, and
@@ -2090,7 +2089,7 @@
}
unsigned NumResults = GetNumNodeResults(Operator, CDP);
- auto Result = llvm::make_unique<TreePatternNode>(Operator, Children, NumResults);
+ TreePatternNode *Result = new TreePatternNode(Operator, Children, NumResults);
Result->setName(OpName);
if (!Dag->getName().empty()) {
@@ -2106,7 +2105,7 @@
/// more type generic things and have useless type casts fold away.
///
/// This returns true if any change is made.
-static bool SimplifyTree(std::unique_ptr<TreePatternNode> &N) {
+static bool SimplifyTree(TreePatternNode *&N) {
if (N->isLeaf())
return false;
@@ -2116,7 +2115,7 @@
N->getExtType(0).isConcrete() &&
N->getExtType(0) == N->getChild(0)->getExtType(0) &&
N->getName().empty()) {
- N.reset(N->getChild(0));
+ N = N->getChild(0);
SimplifyTree(N);
return true;
}
@@ -2124,9 +2123,9 @@
// Walk all children.
bool MadeChange = false;
for (unsigned i = 0, e = N->getNumChildren(); i != e; ++i) {
- std::unique_ptr<TreePatternNode> Child(N->getChild(i));
+ TreePatternNode *Child = N->getChild(i);
MadeChange |= SimplifyTree(Child);
- N->setChild(i, Child.release());
+ N->setChild(i, Child);
}
return MadeChange;
}
@@ -2173,7 +2172,7 @@
// changing the type of the input register in this case. This allows
// us to match things like:
// def : Pat<(v1i64 (bitconvert(v2i32 DPR:$src))), (v1i64 DPR:$src)>;
- if (Nodes[i] == Trees[0].get() && Nodes[i]->isLeaf()) {
+ if (Nodes[i] == Trees[0] && Nodes[i]->isLeaf()) {
DefInit *DI = dyn_cast<DefInit>(Nodes[i]->getLeafValue());
if (DI && (DI->getDef()->isSubClassOf("RegisterClass") ||
DI->getDef()->isSubClassOf("RegisterOperand")))
@@ -2925,27 +2924,26 @@
I->error("Input operand $" + InstInputsCheck.begin()->first +
" occurs in pattern but not in operands list!");
- auto ResultPattern = llvm::make_unique<TreePatternNode>(
- I->getRecord(), ResultNodeOperands,
- GetNumNodeResults(I->getRecord(), *this));
-
+ TreePatternNode *ResultPattern =
+ new TreePatternNode(I->getRecord(), ResultNodeOperands,
+ GetNumNodeResults(I->getRecord(), *this));
// Copy fully inferred output node type to instruction result pattern.
for (unsigned i = 0; i != NumResults; ++i)
ResultPattern->setType(i, Res0Node->getExtType(i));
// Create and insert the instruction.
// FIXME: InstImpResults should not be part of DAGInstruction.
- DAGInsts.insert(std::make_pair(
- I->getRecord(), DAGInstruction(I, Results, Operands, InstImpResults)));
+ DAGInstruction TheInst(I, Results, Operands, InstImpResults);
+ DAGInsts.insert(std::make_pair(I->getRecord(), TheInst));
// Use a temporary tree pattern to infer all types and make sure that the
// constructed result is correct. This depends on the instruction already
// being inserted into the DAGInsts map.
- TreePattern Temp(I->getRecord(), std::move(ResultPattern), false, *this);
+ TreePattern Temp(I->getRecord(), ResultPattern, false, *this);
Temp.InferAllTypes(&I->getNamedNodesMap());
DAGInstruction &TheInsertedInst = DAGInsts.find(I->getRecord())->second;
- TheInsertedInst.setResultPattern(std::move(Temp.getOnlyTree()));
+ TheInsertedInst.setResultPattern(Temp.getOnlyTree());
return TheInsertedInst;
}
@@ -3377,7 +3375,7 @@
InstImpResults);
// Promote the xform function to be an explicit node if set.
- auto DstPattern = std::move(Result.getOnlyTree());
+ TreePatternNode *DstPattern = Result.getOnlyTree();
std::vector<TreePatternNode*> ResultNodeOperands;
for (unsigned ii = 0, ee = DstPattern->getNumChildren(); ii != ee; ++ii) {
TreePatternNode *OpNode = DstPattern->getChild(ii);
@@ -3389,16 +3387,16 @@
}
ResultNodeOperands.push_back(OpNode);
}
- if (!DstPattern->isLeaf()) {
- auto NewPattern = llvm::make_unique<TreePatternNode>(
- DstPattern->getOperator(), ResultNodeOperands,
- DstPattern->getNumTypes());
- for (unsigned i = 0, e = DstPattern->getNumTypes(); i != e; ++i)
- NewPattern->setType(i, DstPattern->getExtType(i));
- DstPattern = std::move(NewPattern);
- }
+ DstPattern = Result.getOnlyTree();
+ if (!DstPattern->isLeaf())
+ DstPattern = new TreePatternNode(DstPattern->getOperator(),
+ ResultNodeOperands,
+ DstPattern->getNumTypes());
- TreePattern Temp(Result.getRecord(), std::move(DstPattern), false, *this);
+ for (unsigned i = 0, e = Result.getOnlyTree()->getNumTypes(); i != e; ++i)
+ DstPattern->setType(i, Result.getOnlyTree()->getExtType(i));
+
+ TreePattern Temp(Result.getRecord(), DstPattern, false, *this);
Temp.InferAllTypes();
@@ -3406,7 +3404,7 @@
PatternToMatch(CurPattern,
CurPattern->getValueAsListInit("Predicates"),
Pattern->getTree(0),
- Temp.getOnlyTree().release(), InstImpResults,
+ Temp.getOnlyTree(), InstImpResults,
CurPattern->getValueAsInt("AddedComplexity"),
CurPattern->getID()));
}