Project

General

Profile

[logo] 
 
Home
News
Activity
About/Contact
Major Tools
  Dinotrace
  Verilator
  Verilog-mode
  Verilog-Perl
Other Tools
  IPC::Locker
  Parallel::Forker
  Voneline
General Info
  Papers

Issue #1491

Add --dpi-hdr-only option

Added by Todd Strader 3 months ago. Updated 3 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Configure/Make/Compiling
% Done:

0%


Description

Towards bug1490. 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.

History

#1 Updated by Todd Strader 3 months ago

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.

#2 Updated by Todd Strader 3 months ago

I added a test for the new option. I believe this is ready to land now: https://github.com/toddstrader/verilator-dev/tree/dpi-header

#3 Updated by Wilson Snyder 3 months ago

  • Category set to Configure/Make/Compiling
  • Status changed from New to Assigned

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.

#4 Updated by Todd Strader 3 months ago

  • Status changed from Assigned to Resolved

Pushed since I've used a similar feature in ModelSim for purposes other than what we're discussing in bug1490.

#5 Updated by Wilson Snyder 3 months ago

  • Status changed from Resolved to Closed

In 4.018.

Also available in: Atom