luke
/
verilator
mirror of https://github.com/verilator/verilator.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Utilize clonep 

Signed-off-by: Ryszard Rozak <rrozak@antmicro.com>
This commit is contained in:
Ryszard Rozak 2026-01-15 14:04:41 +01:00
parent 1cfc7de87d
commit 26736c1156
1 changed files with 4 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
src/V3Force.cpp
View File

@ -490,9 +490,6 @@ public:
}; };
class ForceReplaceVisitor final : public VNVisitor { class ForceReplaceVisitor final : public VNVisitor {
// NODE STATE
// AstNodeExpr::user1 -> bool. Don't visit
// STATE // STATE
const ForceState& m_state; const ForceState& m_state;
AstNodeStmt* m_stmtp = nullptr; AstNodeStmt* m_stmtp = nullptr;
@ -535,7 +532,6 @@ class ForceReplaceVisitor final : public VNVisitor {
} }
void visit(AstSenItem* nodep) override { iterateLogic(nodep); } void visit(AstSenItem* nodep) override { iterateLogic(nodep); }
void visit(AstArraySel* nodep) override { void visit(AstArraySel* nodep) override {
if (nodep->user1()) return;
m_selIndices.push_back(nodep->bitp()); m_selIndices.push_back(nodep->bitp());
iterateChildren(nodep); iterateChildren(nodep);
UASSERT_OBJ(m_selIndices.size(), nodep, "Underflow"); UASSERT_OBJ(m_selIndices.size(), nodep, "Underflow");
@ -569,14 +565,10 @@ class ForceReplaceVisitor final : public VNVisitor {
while (VN_IS(wholeExprp->backp(), NodeExpr)) { while (VN_IS(wholeExprp->backp(), NodeExpr)) {
wholeExprp = VN_AS(wholeExprp->backp(), NodeExpr); wholeExprp = VN_AS(wholeExprp->backp(), NodeExpr);
} }
if (nodep != wholeExprp) { if (wholeExprp != nodep) {
// nodep is a part of a bigger expression AstNodeExpr* const lhsExprp = wholeExprp->cloneTreePure(false);
AstNodeExpr* const copiedWholeExprp = wholeExprp->cloneTreePure(false); nodep->clonep()->replaceWith(lhsp);
copiedWholeExprp->user1(true); lhsp = lhsExprp;
wholeExprp->replaceWith(copiedWholeExprp);
nodep->replaceWith(lhsp);
pushDeletep(nodep);
lhsp = wholeExprp;
} }
std::vector<AstNodeExpr*> reversedIndices(m_selIndices.size()); std::vector<AstNodeExpr*> reversedIndices(m_selIndices.size());
std::reverse_copy(m_selIndices.begin(), m_selIndices.end(), std::reverse_copy(m_selIndices.begin(), m_selIndices.end(),
@ -611,10 +603,6 @@ class ForceReplaceVisitor final : public VNVisitor {
break; break;
} }
} }
void visit(AstNodeExpr* nodep) override {
if (nodep->user1()) return;
iterateChildren(nodep);
}
void visit(AstNode* nodep) override { iterateChildren(nodep); } void visit(AstNode* nodep) override { iterateChildren(nodep); }
public: public: