Move backend compilation utilities #400
Conversation
Some of this could be moved into a more general tools package, but since chisel3 already has a dependency on firrtl ...
Don't push this:
duplicate symbol vl_finish(char const*, int, char const*) in:
top.o
verilated.o
ld: 1 duplicate symbol for architecture x86_64
collect2: error: ld returned 1 exit status
…chisel3 into moveBackendCompilationUtilities
|
I made a Firrtl PR (chipsalliance/firrtl#427) that fixes #452 and added a commit to this to use the new |
|
@ucbjrl @jackkoenig yes, this changes will break the verilog moving annotations. The chisel3 test are obviously not robust enough. It simply checks that a verilog resource makes it into the directory. The code being lost here, looks to see if a list of verilog files file (.f) has been created and if so adds a reference to it on the verilator and vcs command line. I'll make a PR to recover this functionality in firrtl |
|
Thanks @chick. I think it's time to merge this and stop dumping so much junk in /tmp/ |
|
1 run of |
|
@jackkoenig can you give a quick review to firrtl pr #428, to keep things moving forward. |
Move BackendCompilationUtilities into a firrtl util package.
Some of this could be moved into a more general tools package, but since chisel3 already has a dependency on firrtl ...
This should be merged after chipsalliance/firrtl#400 and along with ucb-bar/chisel-testers#72