luke/verilator
1
0
Fork 0
mirror of https://github.com/verilator/verilator.git synced 2026-10-08 02:53:12 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix coverage naming to separate/merge types correctly (#8563)

Browse Source
This commit is contained in:
Marco Bartoli
2026-10-06 12:56:59 -04:00
committed by GitHub
parent dd826d92e1
commit 18cbd53dd1
89 changed files with 4446 additions and 3078 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/V3WidthCommit.cpp
+13 -9
View File
@@ -587,15 +587,19 @@ private:
public:
// CONSTRUCTORS
explicit WidthCommitVisitor(AstNetlist* nodep) {
// Name classes and interfaces while they have their parameter types, as this visit
// moves those to the type table, maybe from another module's types before their own
nodep->foreach([](AstNodeModule* modp) {
if (AstClass* const classp = VN_CAST(modp, Class)) {
classp->dtypeNameFreeze();
} else if (AstIface* const ifacep = VN_CAST(modp, Iface)) {
ifacep->dtypeNameFreeze();
}
});
{
// Name classes and interfaces while they have their parameter types, as this visit
// moves those to the type table, maybe from another module's types before their own,
// with the scopes declaring them indexed, as a class may be named with others
const VDTypeNameScopes scopes{nodep};
nodep->foreach([](AstNodeModule* modp) {
if (AstClass* const classp = VN_CAST(modp, Class)) {
classp->dtypeNameFreeze();
} else if (AstIface* const ifacep = VN_CAST(modp, Iface)) {
ifacep->dtypeNameFreeze();
}
});
}
// Were changing widthMin's, so the table is now somewhat trashed
nodep->typeTablep()->clearCache();
iterate(nodep);