Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign upAdd --dpi-hdr-only option #1491
Comments
This comment has been minimized.
This comment has been minimized.
|
Original Redmine Comment It looks like I can just call compile() twice with different flags. I should be able to make something happen when I can get back to this. |
This comment has been minimized.
This comment has been minimized.
|
Original Redmine Comment I added a test for the new option. I believe this is ready to land now: |
This comment has been minimized.
This comment has been minimized.
|
Original Redmine Comment I'm fine with your squashing and pushing this. However in the larger picture (perhaps worth discussing in the other bug) I'd prefer we not need to run the generation of the dpi headers as a separate step. |
This comment has been minimized.
This comment has been minimized.
|
Original Redmine Comment Pushed since I've used a similar feature in ModelSim for purposes other than what we're discussing in #�. |
This comment has been minimized.
This comment has been minimized.
|
Original Redmine Comment In 4.018. |
Author Name: Todd Strader (@toddstrader)
Original Redmine Issue: 1491 from https://www.veripool.org
Original Assignee: Todd Strader (@toddstrader)
Towards #�. I have the feature coded, but am not sure about the best way to test this:
https://github.com/toddstrader/verilator-dev/tree/dpi-header
Ideally, I'd like to run something like t_dpi_import with and without --dpi-hdr-only (to two different output directories). Then I'd check to make sure that with the flag, the DPI header is the only thing that comes out and make sure it is the same as the file produced without the flag. I'm not sure if driver.pl will support something like this. I need to do some reading to figure that out, but if there are other suggestions for testing I'm all ears.