Skip to content
This repository has been archived by the owner on Aug 20, 2024. It is now read-only.

Multi protobuf module emission and consumption #2344

Merged
merged 25 commits into from
Sep 8, 2021
Merged
Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
38a2786
Multiprotobuf algorithm now pulls the top module from the protobufs
jardhu Aug 16, 2021
4194f86
Remove double assignment compilation error
jardhu Aug 16, 2021
47c3a1b
Add compiler option to emit individual module protobufs
jardhu Aug 23, 2021
abae2b2
Scalafmt
jardhu Aug 23, 2021
7b1817a
Compiler options for proto buf emission
jardhu Aug 24, 2021
2362a75
Implement multi module combination when reading directory of protobufs
jardhu Aug 27, 2021
e2245c0
Use foreacher in protobuf emitter
jardhu Aug 31, 2021
3155a0a
Revert directoryToCircuit change
jardhu Aug 31, 2021
beeec3e
Low opt option now properly uses low opt emitter
jardhu Aug 31, 2021
691994a
emit-modules-protobuf for long -e option
jared-barocsi Sep 1, 2021
3c39745
Directory annotation now correctly throws exceptions
jardhu Sep 1, 2021
0b53842
Dedup collects extmodules if they don't have an accompanying implemen…
jardhu Sep 2, 2021
e161016
Absolute paths for exception references in scala doc
jardhu Sep 2, 2021
c6704d3
Compiler option tests
jardhu Sep 2, 2021
c99ed1e
Fix multiple inclusion of top module in combined circuit
jardhu Sep 2, 2021
f420097
Only one stage is sufficient for multiprotobuf test
jardhu Sep 2, 2021
f991a28
Multi-module circuit combination tests
jardhu Sep 3, 2021
e2189c4
Update src/test/scala/firrtlTests/stage/FirrtlMainSpec.scala
jared-barocsi Sep 3, 2021
c26f7d8
Multi protobuf test checks output circuit against input for equality
jardhu Sep 3, 2021
118ff7d
Move collectInstantiatedModules into private util function
jardhu Sep 4, 2021
0885f70
Comment cleanup
jardhu Sep 7, 2021
4830c89
Rename file -> dir in scaladoc
jardhu Sep 7, 2021
5942386
Apply suggestions from code review
jared-barocsi Sep 8, 2021
fc136d4
Scalafmt
jardhu Sep 8, 2021
ed93404
Merge branch 'master' into multi-protobufs
jackkoenig Sep 8, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
emit-modules-protobuf for long -e option
Co-authored-by: Jack Koenig <koenig@sifive.com>
  • Loading branch information
jared-barocsi and jackkoenig authored Sep 1, 2021
commit 691994a7bce9b3232c4d862d5013188e20ed864d
2 changes: 1 addition & 1 deletion src/main/scala/firrtl/Emitter.scala
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ object EmitAllModulesAnnotation extends HasShellOptions {
helpValueName = Some("<chirrtl|high|middle|low|verilog|mverilog|sverilog>")
),
new ShellOption[String](
longOption = "emit-module-protobufs",
longOption = "emit-modules-protobuf",
toAnnotationSeq = (a: String) =>
a match {
case "chirrtl" =>
Expand Down