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

assign with value containing newline fails #507

Closed
veripoolbot opened this issue May 7, 2012 · 2 comments
Closed

assign with value containing newline fails #507

veripoolbot opened this issue May 7, 2012 · 2 comments
Assignees

Comments

@veripoolbot
Copy link
Collaborator

@veripoolbot veripoolbot commented May 7, 2012


Author Name: Walter Lavino
Original Redmine Issue: 507 from https://www.veripool.org
Original Date: 2012-05-07
Original Assignee: Wilson Snyder (@wsnyder)


Wilson,

The verilog parser would fail in case a white space is found after the 'h'

module any_logic ();

 assign AA_DATA = {8'h00};  // OK

 assign BB_DATA = {8'h

00}; // FAILS

endmodule; // any_logic

Error Message:
%Error: filename.v:6: syntax error, unexpected INTEGER NUMBER, expecting ',' or '}'

regards,
Walter

@veripoolbot

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented May 7, 2012


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-05-07T12:35:36Z


It's the newline, not the spaces.

The parser originally predated Verilog 1995 standard and so needs to see a complete verilog number with optional spaces. For now I fixed newlines, but this ideally requires rework to allow comments and such between the parts, and is related to fixing #�.

Newline part of fix in git towards 3.316.

@veripoolbot

This comment has been minimized.

Copy link
Collaborator Author

@veripoolbot veripoolbot commented Jul 27, 2012


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2012-07-27T22:51:17Z


In 3.316.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.