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

inline literal input for some case #1437

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Conversation

yportne13
Copy link
Contributor

commit 075e654 makes all the literal input not inline. this commit inline literal input for some case

…line. this commit inline literal input for some case
@jijingg
Copy link
Collaborator

jijingg commented Jun 5, 2024

@yportne13 could you add some demo codes so that others can clearly handle what cases

@yportne13
Copy link
Contributor Author

object TestInline extends App {
  SpinalVerilog(new Component{
    val sub = new Component {
      val x = in Bits(1 bits)
    }
    sub.x.setAll()
  })
}

origin:

module unamed_1 (
);

  wire       [0:0]    sub_x;

  unamed sub (
    .x (sub_x)  //i
  );
  assign sub_x = 1'b1;

endmodule

this commit:

module unamed_1 (
);


  unamed sub (
    .x (1'b1)  //i
  );

endmodule

@jijingg jijingg merged commit 8283608 into SpinalHDL:dev Jun 5, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants