From bd628ce5918129ae37b64abf897bf8270c97b11b Mon Sep 17 00:00:00 2001 From: Miodrag Milanovic Date: Thu, 22 Dec 2022 15:26:39 +0100 Subject: Remove deprecated functions --- common/kernel/design_utils.h | 3 --- common/kernel/timing.cc | 8 ++++---- common/kernel/util.h | 19 ------------------- ecp5/pack.cc | 6 +++--- 4 files changed, 7 insertions(+), 29 deletions(-) diff --git a/common/kernel/design_utils.h b/common/kernel/design_utils.h index 069600b5..c290a35b 100644 --- a/common/kernel/design_utils.h +++ b/common/kernel/design_utils.h @@ -31,9 +31,6 @@ NEXTPNR_NAMESPACE_BEGIN Utilities for design manipulation, intended for use inside packing algorithms */ -// Disconnect a net (if connected) from old, and connect it to rep -void replace_port(CellInfo *old_cell, IdString old_name, CellInfo *rep_cell, IdString rep_name); - // If a net drives a given port of a cell matching a predicate (in many // cases more than one cell type, e.g. SB_DFFxx so a predicate is used), return // the first instance of that cell (otherwise nullptr). If exclusive is set to diff --git a/common/kernel/timing.cc b/common/kernel/timing.cc index 47235789..924f336c 100644 --- a/common/kernel/timing.cc +++ b/common/kernel/timing.cc @@ -707,7 +707,7 @@ void TimingAnalyser::print_critical_path(CellPortKey endpoint, domain_id_t domai ctx->getDelayNS(ports.at(cursor).domain_pairs.at(domain_pair).setup_slack)); while (cursor != CellPortKey()) { log(" %s.%s (net %s)\n", ctx->nameOf(cursor.cell), ctx->nameOf(cursor.port), - ctx->nameOf(get_net_or_empty(ctx->cells.at(cursor.cell).get(), cursor.port))); + ctx->nameOf(ctx->cells.at(cursor.cell)->getPort(cursor.port))); if (!ports.at(cursor).arrival.count(dp.key.launch)) break; cursor = ports.at(cursor).arrival.at(dp.key.launch).bwd_max; @@ -865,7 +865,7 @@ struct Timing topological_order.emplace_back(o->net); for (int i = 0; i < clocks; i++) { TimingClockingInfo clkInfo = ctx->getPortClockingInfo(cell.second.get(), o->name, i); - const NetInfo *clknet = get_net_or_empty(cell.second.get(), clkInfo.clock_port); + const NetInfo *clknet = cell.second->getPort(clkInfo.clock_port); IdString clksig = clknet ? clknet->name : async_clock; net_data[o->net][ClockEvent{clksig, clknet ? clkInfo.edge : RISING_EDGE}] = TimingData{clkInfo.clockToQ.maxDelay()}; @@ -1125,7 +1125,7 @@ struct Timing if (portClass == TMG_REGISTER_INPUT) { for (int i = 0; i < port_clocks; i++) { TimingClockingInfo clkInfo = ctx->getPortClockingInfo(usr.cell, usr.port, i); - const NetInfo *clknet = get_net_or_empty(usr.cell, clkInfo.clock_port); + const NetInfo *clknet = usr.cell->getPort(clkInfo.clock_port); IdString clksig = clknet ? clknet->name : async_clock; process_endpoint(clksig, clknet ? clkInfo.edge : RISING_EDGE, clkInfo.setup.maxDelay()); } @@ -1295,7 +1295,7 @@ CriticalPath build_critical_path_report(Context *ctx, ClockPair &clocks, const P if (portClass == TMG_REGISTER_OUTPUT) { for (int i = 0; i < port_clocks; i++) { TimingClockingInfo clockInfo = ctx->getPortClockingInfo(front_driver.cell, front_driver.port, i); - const NetInfo *clknet = get_net_or_empty(front_driver.cell, clockInfo.clock_port); + const NetInfo *clknet = front_driver.cell->getPort(clockInfo.clock_port); if (clknet != nullptr && clknet->name == clocks.start.clock && clockInfo.edge == clocks.start.edge) { last_port = clockInfo.clock_port; clock_start = i; diff --git a/common/kernel/util.h b/common/kernel/util.h index c10abb72..f04a956b 100644 --- a/common/kernel/util.h +++ b/common/kernel/util.h @@ -102,25 +102,6 @@ bool bool_or_default(const Container &ct, const KeyType &key, bool def = false) return bool(int_or_default(ct, key, int(def))); }; -// Return a net if port exists, or nullptr -inline const NetInfo *get_net_or_empty(const CellInfo *cell, const IdString port) -{ - auto found = cell->ports.find(port); - if (found != cell->ports.end()) - return found->second.net; - else - return nullptr; -} - -inline NetInfo *get_net_or_empty(CellInfo *cell, const IdString port) -{ - auto found = cell->ports.find(port); - if (found != cell->ports.end()) - return found->second.net; - else - return nullptr; -} - // Get only value from a forward iterator begin/end pair. // // Generates assertion failure if std::distance(begin, end) != 1. diff --git a/ecp5/pack.cc b/ecp5/pack.cc index 4c8ceee8..f542934d 100644 --- a/ecp5/pack.cc +++ b/ecp5/pack.cc @@ -219,7 +219,7 @@ class Ecp5Packer for (auto &cell : ctx->cells) { CellInfo *ci = cell.second.get(); if (is_ff(ctx, ci)) { - NetInfo *di = get_net_or_empty(ci, id_DI); + NetInfo *di = ci->getPort(id_DI); if (di->driver.cell != nullptr && di->driver.cell->type == id_TRELLIS_COMB && di->driver.port == id_F) { CellInfo *comb = di->driver.cell; if (comb->cluster != ClusterId()) { @@ -306,7 +306,7 @@ class Ecp5Packer // Gets the "COMB1" side of a LUT5, where we pack a LUT[67] into auto get_comb1_from_lut5 = [&](CellInfo *lut5) { - NetInfo *f1 = get_net_or_empty(lut5, id_F1); + NetInfo *f1 = lut5->getPort(id_F1); NPNR_ASSERT(f1 != nullptr); NPNR_ASSERT(f1->driver.cell != nullptr); return f1->driver.cell; @@ -2806,7 +2806,7 @@ bool Arch::pack() void Arch::assign_arch_info_for_cell(CellInfo *ci) { auto get_port_net = [&](CellInfo *ci, IdString p) { - NetInfo *n = get_net_or_empty(ci, p); + NetInfo *n = ci->getPort(p); return n ? n->name : IdString(); }; if (ci->type == id_TRELLIS_COMB) { -- cgit v1.2.3 From 4af8964069c2e250bbbb80851272a9e10ab206f6 Mon Sep 17 00:00:00 2001 From: Miodrag Milanovic Date: Thu, 22 Dec 2022 16:11:10 +0100 Subject: propagate netShareWeight --- common/place/placer_heap.cc | 6 +++++- common/place/placer_heap.h | 1 + 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/common/place/placer_heap.cc b/common/place/placer_heap.cc index 7fa27206..fd722d15 100644 --- a/common/place/placer_heap.cc +++ b/common/place/placer_heap.cc @@ -362,7 +362,11 @@ class HeAPPlacer } else #endif { - if (!placer1_refine(ctx, Placer1Cfg(ctx))) { + auto placer1_cfg = Placer1Cfg(ctx); + placer1_cfg.hpwl_scale_x = cfg.hpwl_scale_x; + placer1_cfg.hpwl_scale_y = cfg.hpwl_scale_y; + placer1_cfg.netShareWeight = cfg.netShareWeight; + if (!placer1_refine(ctx, placer1_cfg)) { return false; } } diff --git a/common/place/placer_heap.h b/common/place/placer_heap.h index e554a8e0..c79e3dfc 100644 --- a/common/place/placer_heap.h +++ b/common/place/placer_heap.h @@ -41,6 +41,7 @@ struct PlacerHeapCfg bool timing_driven; float solverTolerance; bool placeAllAtOnce; + float netShareWeight; bool parallelRefine; int cell_placement_timeout; -- cgit v1.2.3 From 64f7306b24ac26bf05a3009b2f4b54489739ab28 Mon Sep 17 00:00:00 2001 From: Miodrag Milanovic Date: Thu, 22 Dec 2022 20:16:13 +0100 Subject: initialize netShareWeight --- common/place/placer_heap.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/common/place/placer_heap.cc b/common/place/placer_heap.cc index fd722d15..efc04df5 100644 --- a/common/place/placer_heap.cc +++ b/common/place/placer_heap.cc @@ -1817,6 +1817,7 @@ PlacerHeapCfg::PlacerHeapCfg(Context *ctx) criticalityExponent = ctx->setting("placerHeap/criticalityExponent"); timingWeight = ctx->setting("placerHeap/timingWeight"); parallelRefine = ctx->setting("placerHeap/parallelRefine", false); + netShareWeight = ctx->setting("placerHeap/netShareWeight", 0); timing_driven = ctx->setting("timing_driven"); solverTolerance = 1e-5; -- cgit v1.2.3