Reduce the time it takes to figure out we're dealing with an invalid …

…design involving defparams with infinitely recursive module instantiations
MikePopoloski · Nov 26, 2024 · 81a1cfd · 81a1cfd
1 parent 47ccb86
commit 81a1cfd
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/source/ast/Compilation.cpp b/source/ast/Compilation.cpp
@@ -2359,10 +2359,11 @@ void Compilation::resolveDefParamsAndBinds() {
 
         DefParamVisitor initialVisitor(options.maxInstanceDepth, generateLevel);
         initialClone.getRoot(/* skipDefParamsAndBinds */ true).visit(initialVisitor);
-        saveState(initialVisitor, initialClone);
         if (checkProblem(initialVisitor))
             return;
 
+        saveState(initialVisitor, initialClone);
+
         // If we have found more binds, do another visit to let them be applied
         // and potentially add blocks and defparams to our set for this level.
         if (initialClone.bindDirectives.size() > numBindsSeen) {