Conversation
`make gui_synth` is an important use-case for fast
turnaround, less accurate, RTL rinse and repeat cycles
when looking at timing reports, endpoint slack histograms
in the GUI.
The only reason why an .odb file and canonicalized .sdc
file is not always written out by the ORFS by default,
is that this takes a bit of extra time and space and it was
not deemed desirable. It is a bit of a toss-up, accept less
consistency in the flow, or pay a moderate performance
penalty. Switching to always writing .odb and canonicalized
.sdc is simple enough to do if we change our mind and want
simpler, more consistency and fewer suprises across stages.
- gui_synth: improve user experience, same code
as for gui_floorplan/grt, etc. GUI_TIMING=0 now works.
- sta-synth.tcl was trying to do the same as open.tcl. DRY,
now we only have open.tcl
- .odb file output from synth + canonicalization of 1_synth.sdc,
i.e. it is written out by OpenSTA instead of copied from user.
This means that any dependencies that the user's SDC_FILE has
is not dragged into floorplan.
bazel-orfs will be updated to use this, no change to normal
ORFS flow.
To use gui_synth, there is an additional step:
make synth
# New step that writes out 1_synth.odb and canonicalizes
# 1_synth.sdc
make do-1_3_synth
make gui_synth
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
maliberty
approved these changes
Aug 14, 2025
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
make gui_synthis an important use-case for fast turnaround, less accurate, RTL rinse and repeat cycles when looking at timing reports, endpoint slack histograms in the GUI.The only reason why an .odb file and canonicalized .sdc file is not always written out by the synth stage by ORFS by default, is that this takes a bit of extra time and space and it was not deemed desirable. It is a bit of a toss-up, accept less consistency in the flow, or pay a moderate performance penalty. Switching to always writing .odb and canonicalized .sdc is simple enough to do if we change our mind and want simpler, more consistency and fewer suprises across stages.
bazel-orfs will be updated to use this. No change to normal ORFS flow.
To use gui_synth, there is an additional step: