diff options
| author | gatecat <gatecat@ds0.me> | 2021-02-16 11:52:16 +0000 |
|---|---|---|
| committer | gatecat <gatecat@ds0.me> | 2021-02-16 13:31:36 +0000 |
| commit | c7c13cd95f7a25b2c8932ca00ad667ffca381c70 (patch) | |
| tree | 222496d567bd217e6958660a1e1153a1b273ca36 /generic | |
| parent | 815b57b9e1f0c0a7176d146a29cef763bebf343f (diff) | |
| download | nextpnr-c7c13cd95f7a25b2c8932ca00ad667ffca381c70.tar.gz nextpnr-c7c13cd95f7a25b2c8932ca00ad667ffca381c70.tar.bz2 nextpnr-c7c13cd95f7a25b2c8932ca00ad667ffca381c70.zip | |
Remove isValidBelForCell
This Arch API dates from when we were first working out how to
implement placement validity checking, and in practice is little used by
the core parts of placer1/HeAP and the Arch implementation involves a
lot of duplication with isBelLocationValid.
In the short term; placement validity checking is better served by the
combination of checkBelAvail and isValidBelForCellType before placement;
followed by isBelLocationValid after placement (potentially after
moving/swapping multiple cells).
Longer term, removing this API makes things a bit cleaner for a new
validity checking API.
Signed-off-by: gatecat <gatecat@ds0.me>
Diffstat (limited to 'generic')
| -rw-r--r-- | generic/arch.cc | 15 | ||||
| -rw-r--r-- | generic/arch.h | 1 |
2 files changed, 0 insertions, 16 deletions
diff --git a/generic/arch.cc b/generic/arch.cc index 999e5033..a87c4392 100644 --- a/generic/arch.cc +++ b/generic/arch.cc @@ -650,21 +650,6 @@ TimingClockingInfo Arch::getPortClockingInfo(const CellInfo *cell, IdString port return tmg.clockingInfo.at(port).at(index); } -bool Arch::isValidBelForCell(CellInfo *cell, BelId bel) const -{ - std::vector<const CellInfo *> cells; - cells.push_back(cell); - Loc loc = getBelLocation(bel); - for (auto tbel : getBelsByTile(loc.x, loc.y)) { - if (tbel == bel) - continue; - CellInfo *bound = getBoundBelCell(tbel); - if (bound != nullptr) - cells.push_back(bound); - } - return cellsCompatible(cells.data(), int(cells.size())); -} - bool Arch::isBelLocationValid(BelId bel) const { std::vector<const CellInfo *> cells; diff --git a/generic/arch.h b/generic/arch.h index 007478eb..f3a6dccd 100644 --- a/generic/arch.h +++ b/generic/arch.h @@ -357,7 +357,6 @@ struct Arch : ArchAPI<ArchRanges> TimingClockingInfo getPortClockingInfo(const CellInfo *cell, IdString port, int index) const override; bool isValidBelForCellType(IdString cell_type, BelId bel) const override { return cell_type == getBelType(bel); } - bool isValidBelForCell(CellInfo *cell, BelId bel) const override; bool isBelLocationValid(BelId bel) const override; static const std::string defaultPlacer; |