search

chaseruskin / legoHDL

An experimental package manager and development tool for Hardware Description Languages (HDL).
https://c-rus.github.io/legoHDL
MIT License
14 stars 2 forks source link

Adjust graph/export command behavior #15

Closed chaseruskin closed 3 years ago

chaseruskin commented 3 years ago

Design question: switch behavior of explicitly setting top-level to not try to automatically find top-level testbench associated with the design if the design is passed in (user can always pass in tb).

Also, fix if user defines a test bench to be passed in, then that label must be "-SIM-TOP" and not "-SRC-TOP" as currently being defined.

chaseruskin commented 3 years ago

In addition, it has been noted that export does not currently rewrite the top-level and bench metadata fields when the user explicitly defines the top-level. Is this going to be intended behavior?

chaseruskin commented 3 years ago

export and graph behavior is now more defined. By default, a top-level entity will be detected and its test bench as well. Users can explicitly define a top-level to graph or export by passing the entity name. If the entity has a test bench, it will be included by default. Passing the -ignore-tb flag will ignore the design entity's test-bench (this flag may change to -force in the future for more clarity, as ignore-tb has no effect if the explicit entity is already a test-bench).