-
Notifications
You must be signed in to change notification settings - Fork 57
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
Towards release 0.1 #113
Comments
@sbourdeauducq @jordens Any feedback? |
Sounds good to me.
There are also people who will judge the tool based on the quality of the generated Verilog. |
That is indeed true. And I regularly improve Yosys to do better, like in 10e56c7. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
With the core language feature-complete, the build system nearing maturity, and the library gaining many important primitives, it is time to indicate that nMigen can be used in practical designs by people not closely involved in its development. (I've heard that there is a lot of interest in that.) In this issue I outline what I see as prerequisites to milestone 0.1.
Creating signals named like verilog keywords produces invalid verilog #53. We should check for Yosys version inback.verilog
as well, which isShould nMigen check Yosys version? #55.ClockDomain
objects either to convert an isolated module to Verilog or to use the platform with just a single clock domain, it would eliminate a lot of beginner papercuts. That'sFragment.prepare should allow caller to handle nonexistent clock domains #57.Bikeshed: conventions for CDC primitives #97.Now there are some issues that I think shouldn't block 0.1, but I'm open to feedback:
Every other outstanding issue is more of a nice-to-have and shouldn't count here.
The text was updated successfully, but these errors were encountered: