Project

General

Profile

[logo] 
 
Home
News
Activity
About/Contact
Major Tools
  Dinotrace
  Verilator
  Verilog-mode
  Verilog-Perl
Other Tools
  BugVise
  CovVise
  Force-Gate-Sim
  Gspice
  IPC::Locker
  Rsvn
  SVN::S4
  Voneline
  WFH
General Info
  Papers

Issue #1491

Add --dpi-hdr-only option

Added by Todd Strader 29 days ago. Updated 23 days 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 29 days 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 28 days 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 25 days 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 25 days 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 23 days ago

  • Status changed from Resolved to Closed

In 4.018.

Also available in: Atom