Coverage Declarations does not honor output split cfuncs #1098
Labels
resolution: fixed
Closed; fixed
Comments
|
Original Redmine Comment Sorry, sent the wrong commithash. This is the correct one: |
|
Original Redmine Comment 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. |
|
Original Redmine Comment In 3.890. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Author Name: Johan Bjork
Original Redmine Issue: 1098 from https://www.veripool.org
Original Assignee: Johan Bjork
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: phb/verilator-dev@ad106e4
The text was updated successfully, but these errors were encountered: