diff options
| author | Kevin Laeufer | 2020-07-17 18:07:54 -0700 |
|---|---|---|
| committer | GitHub | 2020-07-18 01:07:54 +0000 |
| commit | 1b9f4ddff4102fee72ae4dd8c111c82c32e42d5d (patch) | |
| tree | fee6379c83e4026edcf3d577a2a9474024ed0b59 /scripts/parse_firrtl_transform_log.py | |
| parent | 5f70175d24cbeeef2ffae3fb00b99e06c5462bd0 (diff) | |
Faster dedup instance graph (#1732)
* dedup: add faster InstanceGraph implementation and use it in dedup
The new implementation takes care not to hash the instance
types contained in DefInstance nodes.
This should make dedup considerably faster.
* FastInstanceGraph: cache vertices for faster findInstancesInHierarchy
* FastInstanceGraph: remove the parent name field since it isn't actually necessary
* FastInstanceGraph -> InstanceKeyGraph
* InstanceGraph: describe performance problems.
* InstanceKeyGraph: turn moduleMap into a def instead of a val
This will make changing implementation details much easier
in the future.
* InstanceKeyGraph: return childInstances as Seq instead of Map
This ensures a deterministic iteration order and it
can easily be turned into a Map for O(1) accesses.
* InstanceKeyGraph: add tests for public methods
* InstanceKeyGraph: group public methods together
* InstanceKeyGraphSpec: fix wording of a comment
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Diffstat (limited to 'scripts/parse_firrtl_transform_log.py')
0 files changed, 0 insertions, 0 deletions
