Skip to content

Commit d425813

Browse files
committed
Disable sizedown move until ERC checking is added.
Signed-off-by: Matthew Guthaus <[email protected]>
1 parent 41421de commit d425813

39 files changed

+588
-520
lines changed

src/rsz/README.md

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ set_wire_rc
6060
| Switch Name | Description |
6161
| ----- | ----- |
6262
| `-clock` | Enable setting of RC for clock nets. |
63-
| `-signal` | Enable setting of RC for signal nets. |
63+
| `-signal` | Enable setting of RC for signal nets. |
6464
| `-layers` | Use the LEF technology resistance and area/edge capacitance values for the layers. The values for each layers will be used for wires with the prefered layer direction, if 2 or more layers have the same prefered direction the avarege value is used for wires with that direction. This is used for a default width wire on the layer. |
6565
| `-layer` | Use the LEF technology resistance and area/edge capacitance values for the layer. This is used for a default width wire on the layer. |
6666
| `-resistance` | Resistance per unit length, units are from the first Liberty file read. |
@@ -70,7 +70,6 @@ set_wire_rc
7070
| `-v_resistance` | Resistance per unit length for vertical wires, units are from the first Liberty file read. |
7171
| `-v_capacitance` | Capacitance per unit length for vertical wires, units are from the first Liberty file read. |
7272

73-
7473
### Set Layer RC
7574

7675
The `set_layer_rc` command can be used to set the resistance and capacitance
@@ -110,7 +109,7 @@ After the `global_route` command has been called, the global routing topology
110109
and layers can be used to estimate parasitics with the `-global_routing`
111110
flag.
112111

113-
The optional argument `-spef_file` can be used to write the estimated parasitics using
112+
The optional argument `-spef_file` can be used to write the estimated parasitics using
114113
Standard Parasitic Exchange Format.
115114

116115
```tcl
@@ -249,7 +248,7 @@ the wire. It also resizes gates to normalize slews. Use `estimate_parasitics
249248
-placement` before `repair_design` to estimate parasitics considered
250249
during repair. Placement-based parasitics cannot accurately predict
251250
routed parasitics, so a margin can be used to "over-repair" the design
252-
to compensate.
251+
to compensate.
253252

254253
```tcl
255254
repair_design
@@ -318,7 +317,7 @@ Setup repair is done before hold repair so that hold repair does not
318317
cause setup checks to fail.
319318

320319
The worst setup path is always repaired. Next, violating paths to
321-
endpoints are repaired to reduced the total negative slack.
320+
endpoints are repaired to reduced the total negative slack.
322321

323322
```tcl
324323
repair_timing
@@ -356,7 +355,7 @@ repair_timing
356355
| `-setup_margin` | Add additional setup slack margin. |
357356
| `-hold_margin` | Add additional hold slack margin. |
358357
| `-allow_setup_violations` | While repairing hold violations, buffers are not inserted that will cause setup violations unless `-allow_setup_violations` is specified. |
359-
| `-sequence` | Specify a particular order of setup timing optimizations. The default is "unbuffer,sizedown,sizeup,swap,buffer,clone,split". Obeys skip flags also. |
358+
| `-sequence` | Specify a particular order of setup timing optimizations. The default is "unbuffer,sizeup,swap,buffer,clone,split". Obeys skip flags also. |
360359
| `-skip_pin_swap` | Flag to skip pin swap. The default is to perform pin swap transform during setup fixing. |
361360
| `-skip_gate_cloning` | Flag to skip gate cloning. The default is to perform gate cloning transform during setup fixing. |
362361
| `-skip_size_down` | Flag to skip gate down sizing. The default is to perform non-critical fanout gate down sizing transform during setup fixing. |
@@ -372,7 +371,7 @@ repair_timing
372371

373372
Use`-recover_power` to specify the percent of paths with positive slack which
374373
will be considered for gate resizing to save power. It is recommended that
375-
this option be used with global routing based parasitics.
374+
this option be used with global routing based parasitics.
376375

377376
#### Instance Name Prefixes
378377

@@ -609,9 +608,9 @@ report_wns
609608

610609
## Regression tests
611610

612-
There are a set of regression tests in `./test`. For more information, refer to this [section](../../README.md#regression-tests).
611+
There are a set of regression tests in `./test`. For more information, refer to this [section](../../README.md#regression-tests).
613612

614-
Simply run the following script:
613+
Simply run the following script:
615614

616615
```shell
617616
./test/regression

src/rsz/src/RepairSetup.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ bool RepairSetup::repairSetup(const float setup_slack_margin,
139139
if (!skip_buffer_removal) {
140140
move_sequence.push_back(resizer_->unbuffer_move);
141141
}
142-
move_sequence.push_back(resizer_->size_down_move);
142+
// TODO: Add size_down_move to the sequence if we want to allow
143143
// Always have sizing
144144
move_sequence.push_back(resizer_->size_up_move);
145145
if (!skip_pin_swap) {

src/rsz/test/buffer_varying_lengths.ok

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ Iteration | Area | Resized | Buffers | Nets repaired | Remaining
1010
[INFO RSZ-0038] Inserted 161 buffers in 118 nets.
1111
worst slack max -0.279
1212
tns max -30.987
13-
[INFO RSZ-0100] Repair move sequence: UnbufferMove SizeDownMove SizeUpMove SwapPinsMove BufferMove CloneMove SplitLoadMove
13+
[INFO RSZ-0100] Repair move sequence: UnbufferMove SizeUpMove SwapPinsMove BufferMove CloneMove SplitLoadMove
1414
[INFO RSZ-0094] Found 200 endpoints with setup violations.
1515
[INFO RSZ-0099] Repairing 200 out of 200 (100.00%) violating endpoints...
1616
Iter | Removed | Resized | Inserted | Cloned | Pin | Area | WNS | TNS | Viol | Worst

src/rsz/test/clone_flat.ok

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[INFO ODB-0227] LEF file: Nangate45/Nangate45.lef, created 22 layers, 27 vias, 135 library cells
22
worst slack max -0.16
3-
[INFO RSZ-0100] Repair move sequence: UnbufferMove SizeDownMove SizeUpMove SwapPinsMove BufferMove CloneMove SplitLoadMove
3+
[INFO RSZ-0100] Repair move sequence: UnbufferMove SizeUpMove SwapPinsMove BufferMove CloneMove SplitLoadMove
44
[INFO RSZ-0094] Found 150 endpoints with setup violations.
55
[INFO RSZ-0099] Repairing 150 out of 150 (100.00%) violating endpoints...
66
Iter | Removed | Resized | Inserted | Cloned | Pin | Area | WNS | TNS | Viol | Worst

0 commit comments

Comments
 (0)