Skip to content

Commit b6b55e8

Browse files
committed
rsz: Appease the clang-tidy gods
Signed-off-by: Martin Povišer <[email protected]>
1 parent 8b47790 commit b6b55e8

File tree

4 files changed

+16
-13
lines changed

4 files changed

+16
-13
lines changed

src/rsz/src/BufferedNet.cc

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,10 @@
3333
#include "grt/RoutePt.h"
3434
#include "sta/Hash.hh"
3535

36+
namespace sta {
37+
class Port;
38+
}
39+
3640
namespace rsz {
3741

3842
using sta::Port;
@@ -950,12 +954,11 @@ BufferedNetPtr Resizer::makeBufferedNetGroute(const Pin* drvr_pin,
950954
}
951955
}
952956
return bnet;
953-
} else {
954-
logger_->warn(RSZ,
955-
73,
956-
"driver pin {} not found in global routes",
957-
db_network_->pathName(drvr_pin));
958957
}
958+
logger_->warn(RSZ,
959+
73,
960+
"driver pin {} not found in global routes",
961+
db_network_->pathName(drvr_pin));
959962
return nullptr;
960963
}
961964

src/rsz/src/Graphics.cc

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33

44
#include "Graphics.hh"
55

6+
#include "BufferedNet.hh"
67
#include "gui/gui.h"
78
#include "odb/db.h"
89
#include "odb/geom.h"

src/rsz/src/Rebuffer.cc

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -603,9 +603,8 @@ static std::optional<int> findWireLayer(BnetPtr node)
603603
}
604604
if (node->type() == BnetType::wire) {
605605
return {node->layer()};
606-
} else {
607-
return {};
608606
}
607+
return {};
609608
}
610609

611610
// Find initial timing-optimized buffering choice over the provided tree

src/rsz/src/RepairDesign.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1408,7 +1408,7 @@ void RepairDesign::repairNetVia(const BufferedNetPtr& bnet,
14081408
bnet->setFanout(bnet->ref()->fanout());
14091409
float r_via = bnet->viaResistance(corner_, resizer_, estimate_parasitics_);
14101410
bnet->setMaxLoadSlew(bnet->ref()->maxLoadSlew()
1411-
- r_via * bnet->ref()->cap() * slew_rc_factor_);
1411+
- (r_via * bnet->ref()->cap() * slew_rc_factor_));
14121412
}
14131413

14141414
void RepairDesign::repairNetWire(
@@ -1494,7 +1494,7 @@ void RepairDesign::repairNetWire(
14941494
bnet->setCapacitance(load_cap);
14951495
bnet->setFanout(bnet->ref()->fanout());
14961496
bnet->setMaxLoadSlew(bnet->ref()->maxLoadSlew()
1497-
- r_wire * (c_wire / 2 + ref_cap) * slew_rc_factor_);
1497+
- (r_wire * (c_wire / 2 + ref_cap) * slew_rc_factor_));
14981498

14991499
//============================================================================
15001500
// Back up from pt to from_pt adding repeaters as necessary for
@@ -1556,8 +1556,8 @@ void RepairDesign::repairNetWire(
15561556
//
15571557
// We solve a quadratic eq. to find the maximum conforming length.
15581558
float a = wire_res * wire_cap / 2;
1559-
float b = r_drvr * wire_cap + wire_res * ref_cap;
1560-
float c = r_drvr * ref_cap - max_load_slew_margined / slew_rc_factor_;
1559+
float b = (r_drvr * wire_cap) + (wire_res * ref_cap);
1560+
float c = (r_drvr * ref_cap) - (max_load_slew_margined / slew_rc_factor_);
15611561
float l = (-b + sqrt(b * b - 4 * a * c)) / (2 * a);
15621562
if (l >= 0.0) {
15631563
split_length = min(split_length, metersToDbu(l));
@@ -1628,8 +1628,8 @@ void RepairDesign::repairNetWire(
16281628

16291629
bnet->setCapacitance(load_cap);
16301630
bnet->setFanout(repeater_fanout);
1631-
bnet->setMaxLoadSlew(max_load_slew
1632-
- r_wire * (c_wire / 2 + ref_cap) * slew_rc_factor_);
1631+
bnet->setMaxLoadSlew(
1632+
max_load_slew - (r_wire * (c_wire / 2 + ref_cap) * slew_rc_factor_));
16331633

16341634
debugPrint(logger_,
16351635
RSZ,

0 commit comments

Comments
 (0)