Skip to content

Commit d233581

Browse files
authored
Make wire_load_tree_ default to unknown so we can detect when it's not present in a Liberty file. (#402)
This is potentially a behavior change, but I think omitting this is rare. I've only seen it in some DTCD Liberty files. In those Liberty libraries, it seems to be expected that the Liberty data is valid for all `WireloadTree` types. Thus it is necessary to distinguish between "wire load tree was specified as 'balanced'" and "wire load tree was not specified".
1 parent c387621 commit d233581

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

liberty/Liberty.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3042,7 +3042,7 @@ OperatingConditions::OperatingConditions(const char *name) :
30423042
Pvt(0.0, 0.0, 0.0),
30433043
name_(name),
30443044
// Default wireload tree.
3045-
wire_load_tree_(WireloadTree::balanced)
3045+
wire_load_tree_(WireloadTree::unknown)
30463046
{
30473047
}
30483048

parasitics/Parasitics.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -257,12 +257,12 @@ Parasitics::makeWireloadNetwork(const Pin *drvr_pin,
257257
makeWireloadNetworkWorst(parasitic, drvr_pin, net, wireload_cap,
258258
wireload_res, fanout);
259259
break;
260+
case WireloadTree::unknown:
260261
case WireloadTree::balanced:
261262
makeWireloadNetworkBalanced(parasitic, drvr_pin, wireload_cap, wireload_res,
262263
fanout);
263264
break;
264265
case WireloadTree::best_case:
265-
case WireloadTree::unknown:
266266
makeWireloadNetworkBest(parasitic, drvr_pin, wireload_cap, wireload_res,
267267
fanout);
268268
break;

0 commit comments

Comments
 (0)