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

symbol_search(): Track object visibility directly 

The scope traversal tracks whether it has crossed a module boundary and
uses the inverse of that state to decide whether objects are visible.

Track object visibility directly instead. This keeps the existing behavior
while making the module-boundary rule and later traversal changes easier to
follow.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
This commit is contained in:
Lars-Peter Clausen 2026-08-15 21:00:12 -07:00
parent 6155d56482
commit 4e425289bb
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
symbol_search.cc
View File

@ -388,7 +388,7 @@ bool symbol_search(const LineInfo*li, Design*des, NetScope*scope,
}
}
bool passed_module_boundary = false;
bool search_objects = true;
// At this point, we've stripped right-most components until the search
// found the scope part of the path, or there is no scope part of the
@ -427,7 +427,7 @@ bool symbol_search(const LineInfo*li, Design*des, NetScope*scope,
// ... not_ok; // <-- Should NOT match.
// ... top.not_ok; // Matches.
// endmodule
if (!passed_module_boundary) {
if (search_objects) {
scope_object_search_result_t object_result =
symbol_search_scope_objects(
li, des, scope, start_scope, path, path_tail,
@ -479,10 +479,10 @@ bool symbol_search(const LineInfo*li, Design*des, NetScope*scope,
// If there is no prefix, then we are free to scan upwards looking
// for a scope name. Note that only scopes can be searched for up
// past module boundaries. To handle that, set a flag to indicate
// that we passed a module boundary on the way up.
// past module boundaries. Stop searching for objects after leaving
// the module, but continue searching for matching scope names.
if (scope->type()==NetScope::MODULE && !scope->nested_module())
passed_module_boundary = true;
search_objects = false;
scope = scope->parent();
@ -503,7 +503,7 @@ bool symbol_search(const LineInfo*li, Design*des, NetScope*scope,
if (scope == 0 && start_scope != 0) {
scope = start_scope->unit();
start_scope = 0;
passed_module_boundary = false;
search_objects = true;
}
}