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

Chipyard 1.3 Release #500

Merged
merged 198 commits into from
Jun 1, 2020
Merged

Chipyard 1.3 Release #500

merged 198 commits into from
Jun 1, 2020

Conversation

alonamid
Copy link
Contributor

@alonamid alonamid commented Mar 28, 2020

Developers: Please add descriptions of your changes to this PR as your PRs are merged into Dev

Added

Changed

Fixed

Deprecated

Removed

davidbiancolin and others added 26 commits November 1, 2019 17:17
Still need to work on the asm-test/benchmark integration
Also begin power integration
Target-Facing Support for Multiclock Simulation in FireSim
abejgonzalez and others added 3 commits March 30, 2020 17:04
* [make] split up specific make vars/targets into frags

* [make] move dramsim and max-cycles into SIM_FLAGS

* [misc] move ariane configs to configs/ folder
This adds an additional layer (ChipTop) between the System module and the TestHarness. The IOBinder API is now changed to take only a single parameter (an Any) and return a 3 things: The IO port(s), the IO cell(s), and a function to call inside the test harness, which is analogous to the old IOBinder function, except that it takes a TestHarness object as an argument instead of (clock, reset, success).
* A new Top-level module, ChipTop, has been created. ChipTop instantiates a "system" module specified by BuildSystem.
* BuildTop now builds a ChipTop dut module in the TestHarness by default
* A new BuildSystem key has been added, which by default builds DigitalTop (previously just called Top)
* The IOBinders API has changed. IOBinders are now called inside of ChipTop and return a tuple3 of (IO ports, IO cells, harness functions). The harness functions are now called inside the TestHarness (this is analogous to the previous IOBinder functions).
* IO cell models have been included in ChipTop. These can be replaced with real IO cells for tapeout, or used as-is for simulation.
* The default for the TOP make variable is now ChipTop (was Top)
abejgonzalez and others added 16 commits May 28, 2020 13:50
Fix small issues with hammer-sim dut name and flags.

It would be cool to only build dramsim when our config includes
dramsim but this matches the way the other simulators handle this
in chipyard.
We also strip out empty CC flags because hammer doesn't prune these.
Its unclear whether the hammer should handle this itself or not,
if it eventually does we can remove this check.
Add dramsim_lib as a dependency of hammer sims
Symlink dramsim2 into hammer-sims rundirs
update some hammer generated makefile targets that were missed before
bump barstools to get an iocell fix
More fixes for post-syn/par hammer simulations
@alonamid alonamid changed the title [DO NOT MERGE] Dev->Master Tracking PR [DO NOT MERGE] Chipyard 1.3 Release May 30, 2020
@abejgonzalez abejgonzalez changed the title [DO NOT MERGE] Chipyard 1.3 Release Chipyard 1.3 Release Jun 1, 2020
Copy link
Contributor

@abejgonzalez abejgonzalez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@davidbiancolin davidbiancolin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • 10K / - 1K. Damn.

Ship it!

@abejgonzalez abejgonzalez merged commit c576a7e into master Jun 1, 2020
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.