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

Feature request: synth* to call check after hierarchy #429

Open
Laksen opened this issue Oct 1, 2017 · 0 comments
Open

Feature request: synth* to call check after hierarchy #429

Laksen opened this issue Oct 1, 2017 · 0 comments
Assignees
Labels
discuss to be discussed at next dev jour fixe (see #devel-discuss at https://yosyshq.slack.com/) feature-request

Comments

@Laksen
Copy link

Laksen commented Oct 1, 2017

The below test module is very obviously broken since the test2 instance is wired wrongly. Running a hierarchy, and then check gives good feedback, but just a hierarchy; flatten; will make all of the warnings disappear and subtle bugs sneak in.

module test(input x,
            output y);

  test2 instance(
    .y(x),
    .x(y)
  );

endmodule

module test2(input x,
             output y);

  assign x = y;

endmodule

Basically this is a request for having the inbuilt synth scripts use check after hierarchy to expose warnings that might otherwise be lost.

@cliffordwolf cliffordwolf self-assigned this Oct 3, 2017
@lethalbit lethalbit added the discuss to be discussed at next dev jour fixe (see #devel-discuss at https://yosyshq.slack.com/) label Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discuss to be discussed at next dev jour fixe (see #devel-discuss at https://yosyshq.slack.com/) feature-request
Projects
None yet
Development

No branches or pull requests

4 participants