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 #1098

Coverage Declarations does not honor output split cfuncs

Added by Johan Bjork 5 months ago. Updated 4 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
% Done:

0%


Description

Hey Wilson,

Changed V3EmitC to call a predefined function for initializing coverage and moved that function creation to the same place as where the variable resets are being created. This makes it possible to compile large projects (w/ heavy inlining) with coverage. (Without this patch the C++ would never finish due to the ::configure() method being way too large)

Patch here: https://github.com/phb/verilator-dev/commit/ad106e47fdbfa2a67574a0b0b174eb7cc79c8e96

History

#1 Updated by Johan Bjork 5 months ago

Sorry, sent the wrong commithash. This is the correct one:

https://github.com/phb/verilator-dev/commit/92e6c4941200931ce3c676d348255c1530bf842b

#2 Updated by Wilson Snyder 5 months ago

  • Status changed from New to Resolved

Applied to git towards 3.889.

All looked good thanks!

My only change was going to be to rename the visitor to the new standard, then realized the name just seemed too long to me (V3CConstructorsVisitor). I'm not really in love with how I renamed the file to make it feel better either (V3CCtors) but it'll do.

#3 Updated by Wilson Snyder 4 months ago

  • Status changed from Resolved to Closed

In 3.890.

Also available in: Atom