Added description of inserted inst name prefixes #7213
Conversation
|
clang-tidy review says "All clean, LGTM! 👍" |
|
@povik @precisionmoon please review for accuracy and completeness |
povik
left a comment
povik
left a comment
There was a problem hiding this comment.
Looks complete and accurate in so far as the rsz part except for comments
|
@jeffng-or , thanks much for documenting the name prefixes. This definitely improves tool usability. |
Incorporated review feedback Signed-off-by: Jeff Ng <[email protected]>
jeffng-or
force-pushed
the
inserted-inst-doc-addition
branch
from
02e495c to
f6a20b4
Compare
|
Thanks for the feedback. I've rev'd the rsz README |
|
clang-tidy review says "All clean, LGTM! 👍" |
| | load_slew | Fixing max transition violations | | ||
| | max_cap | Fixing max capacitance | | ||
| | max_length | Fixing max length | | ||
| | wire | Repairs load slew, length, and max capacitance violations in net wire segment | |
There was a problem hiding this comment.
We have prefixes each of these separately above. How does wire differ?
There was a problem hiding this comment.
Based on my read of RepairDesign.cc, wire is only used for the BufferedNetType::wire. The others are used for BufferedNetType::junction. wire is then used to address length, max_cap, or load_slew violations.
|
@maliberty, OK to merge? |
4 participants
Added description for inst name prefixes used when rsz & cts insert buffers.