Skip to content

Commit c78b62c

Browse files
committed
Use InstanceInfo
1 parent 9d25120 commit c78b62c

File tree

5 files changed

+58
-405
lines changed

5 files changed

+58
-405
lines changed

include/circt/Dialect/FIRRTL/InstanceChoiceInfo.h

Lines changed: 0 additions & 143 deletions
This file was deleted.

include/circt/Dialect/FIRRTL/Passes.td

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -921,18 +921,12 @@ def PopulateInstanceChoiceSymbols : Pass<"firrtl-populate-instance-choice-symbol
921921
}
922922

923923
def CheckInstanceChoice : Pass<"firrtl-check-instance-choice", "firrtl::CircuitOp"> {
924-
let summary = "Check for nested instance choices with different options";
924+
let summary = "Check for nested instance choices";
925925
let description = [{
926-
This pass checks that no path from a public module contains nested instance
927-
choices with different options (conjunction is banned). On any single path,
928-
you can only go through instance choices of the same option. However,
929-
reaching the same module through different options on different paths is
930-
allowed (disjunction is OK).
926+
This pass checks that there are no nested instance choices in the design.
927+
A module that is instantiated via an instance choice operation cannot
928+
contain any instance choice operations itself.
931929
}];
932-
let options = [
933-
Option<"dumpInfo", "dump-info", "bool", "false",
934-
"Dump instance choice information for testing">
935-
];
936930
}
937931

938932
def InferDomains : Pass<"firrtl-infer-domains", "firrtl::CircuitOp"> {

0 commit comments

Comments
 (0)