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 /common/nextpnr.h | |
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 'common/nextpnr.h')
-rw-r--r-- | common/nextpnr.h | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/common/nextpnr.h b/common/nextpnr.h index b6ee33fe..4ddf8fef 100644 --- a/common/nextpnr.h +++ b/common/nextpnr.h @@ -1169,7 +1169,6 @@ template <typename R> struct ArchAPI : BaseCtx virtual BelBucketId getBelBucketByName(IdString name) const = 0; virtual BelBucketId getBelBucketForBel(BelId bel) const = 0; virtual BelBucketId getBelBucketForCellType(IdString cell_type) const = 0; - virtual bool isValidBelForCell(CellInfo *cell, BelId bel) const = 0; virtual bool isBelLocationValid(BelId bel) const = 0; virtual typename R::CellTypeRangeT getCellTypes() const = 0; virtual typename R::BelBucketRangeT getBelBuckets() const = 0; @@ -1420,7 +1419,6 @@ template <typename R> struct BaseArch : ArchAPI<R> { return getBelBucketByName(cell_type); }; - virtual bool isValidBelForCell(CellInfo *cell, BelId bel) const override { return true; } virtual bool isBelLocationValid(BelId bel) const override { return true; } virtual typename R::CellTypeRangeT getCellTypes() const override { |