Add explicit ports #358
Merged
Add explicit ports #358
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
this commit also: - add helpfull errors on missing packages - add facility to only run subset of task - fix the overrides of config files, must specify that we want to override - add method to forcefully disable coloring in case we can't pick up that its a CI
Previous changes would avoid absorbing buffers driving two primary outputs. We now absorb buffers in that case.
Less conservative buffer absorption improves delay on strong_multiclock test, as an additional buffer is absorbed.
Previously would crash. Now works and draws blocks the color of the physical tile currently implementing them. Note that internal hierarchy drawing still needs work.
Previously only the first block type's internals were drawn.
Previously this was incorrectly skipped. Also fixes range on GUI selector.
This avoids significantly bloating the auto-sized device if a logical block type (e.g. LAB) can be implemented at multiple sites (e.g. LAB or MLAB).
…tions Initial placement now tries all equivalent tile locations before giving up when placing macros.
The drawing code now directly looks-up the default block color based on the placement state. This avoids having to synchronize it manually and fixes a crash when graphics is turned off.
New format includes enough context to parse as a single line
probot-autolabeler
bot
added
lang-cpp
lang-shell
libarchfpga
Odin
scripts
tests
VPR
labels
Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Alessandro Comodi <[email protected]>
This implements part one of verilog-to-routing#1063 Signed-off-by: Keith Rothman <[email protected]>
This reverts commit 502be5a.
This reverts commit 5007035.
The heap approximation doesn't precisely follow the heap property, but offers an approximation that is sufficent for the router's purpose. This new data structure is faster to clear, faster during route time, and results in better router behavior because it provides some randomness on elements with costs within ~1%. Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Alessandro Comodi <[email protected]>
Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Andrew Butt <[email protected]>
This helps avoid conflicts with other changes. Signed-off-by: Keith Rothman <[email protected]>
This avoids width * height A* runs, and is likely much faster and will return better results than the previous place delay matrix algorithm for larger graphs. Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Keith Rothman <[email protected]> Updated README.md Signed-off-by: Alessandro Comodi <[email protected]>
Signed-off-by: Keith Rothman <[email protected]>
- Use is_empty_type over direct comparisions. Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Keith Rothman <[email protected]>
Signed-off-by: Keith Rothman <[email protected]>
…ip/avoid-criticality-issue', 'wip/avoid_unordered_set', 'wip/disable-check-route-option', 'wip/faster_rr_graph_read', 'wip/inline_main_loop', 'wip/length_bounded', 'wip/lookahead_sampling2', 'wip/place-constraints', 'wip/refactor_heap2', 'wip/revert_clock_propigation', 'wip/rr-graph-binary-read-write', 'wip/symbiflow-badger' and 'wip/use_expansion_for_place_delay' into add_explicit_ports
This is an Octopus Merge commit of the following branches: wip/add-issue-template wip/add_explicit_ports wip/avoid-criticality-issue wip/avoid_unordered_set wip/disable-check-route-option wip/faster_rr_graph_read wip/inline_main_loop wip/length_bounded wip/lookahead_sampling2 wip/place-constraints wip/refactor_heap2 wip/revert_clock_propigation wip/rr-graph-binary-read-write wip/symbiflow-badger wip/use_expansion_for_place_delay Signed-off-by: Keith Rothman <[email protected]>
litghost
force-pushed
the
add_explicit_ports
branch
from
3061207 to
be9a50d
Compare
|
I'm expecting the VTR strong tests to fail because of QoR. I'll update this PR with a QoR update once I've isolated the failing QoR's. |
Huh, strong tests passed QoR. Okay, ready for review! |
Closed
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Also updates to latest upstream VPR.
See: #357 for the changes from the new wip branch only