Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Duplicate declaration of cell" diagnostic with missing location #1381

Closed
veripoolbot opened this issue Dec 31, 2018 · 2 comments
Closed

"Duplicate declaration of cell" diagnostic with missing location #1381

veripoolbot opened this issue Dec 31, 2018 · 2 comments
Assignees

Comments

@veripoolbot
Copy link

@veripoolbot veripoolbot commented Dec 31, 2018


Author Name: Al Grant
Original Redmine Issue: 1381 from https://www.veripool.org

Original Assignee: Wilson Snyder (@wsnyder)


Compiling a source file with declarations but no top module:

logic a;

results in an error with no location:

%Error: Duplicate declaration of cell: $unit
%Error: ... Location of original declaration
%Error: Exiting due to 1 error(s)

A diagnostic about no module definition might be reasonable, but this looks like Verilator has internally tried to do something twice that it should only do once and failed a self-check. A duplicate declaration diagnostic without locations for the declarations seems like a misdiagnosis.

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Jan 2, 2019


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2019-01-02T23:40:04Z


No reason for an error at all in this case, as code is legal.

Fixed in git towards 4.010.

@veripoolbot

This comment has been minimized.

Copy link
Author

@veripoolbot veripoolbot commented Jan 28, 2019


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2019-01-28T12:32:25Z


In 4.010.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.