Verilator

09/03/2019

11:37 PM Issue #1499: Have V3EmitC::emitcSyms honor --output-split-cfuncs

It just dawned on me that I didn't modify t_flag_csplit.pl to verify that it is now producing Syms__#.cpp files (whic... Todd Strader

08:33 PM Issue #1499: Have V3EmitC::emitcSyms honor --output-split-cfuncs

Proposed change:
https://github.com/toddstrader/verilator-dev/tree/sym-split Todd Strader

05:16 PM Issue #1499 (Closed): Have V3EmitC::emitcSyms honor --output-split-cfuncs

I'm planning to modify V3EmitC::emitcSyms to respect --output-split-cfuncs in order to break up the symbol object con... Todd Strader

12:42 PM Issue #1496: Support additional file I/O functions $fseek, $ftell, $frewind

I agree on the certificate and ack the request explicitly. Howard Su

10:50 AM Issue #1496 (Assigned): Support additional file I/O functions $fseek, $ftell, $frewind

Excellent work, even in updating the tests.
Just one legal thing, in the patch please insert your name in docs/CON... Wilson Snyder

04:14 AM Issue #1496 (Closed): Support additional file I/O functions $fseek, $ftell, $frewind

Attached the patch to add three functions.
The scenario is that my test bench is able to parsing elf file and load... Howard Su

12:23 PM Issue #1369: Raise error / warning on continous assignment to reg

Git is the change-by-change repo, which is snapshotted for the tarballs.
Anyhow the version released this weekend ... Wilson Snyder

12:06 PM Issue #1369: Raise error / warning on continous assignment to reg

Is the git version different from tarball version ?I thought that they were the same.
I'll have to use git version. ... Kris Jeon

11:55 AM Issue #1369: Raise error / warning on continous assignment to reg

See https://www.veripool.org/projects/verilator/wiki/Installing
Wilson Snyder

11:23 AM Issue #1369: Raise error / warning on continous assignment to reg

Oh, where can I get the unreleased git version ?
Kris Jeon

11:05 AM Issue #1490: Add an option to create a DPI protected library

I'm unaware of how Modelsim handles DPI.
Was thinking the structure would be part of the AST (as everything tries ... Wilson Snyder

11:00 AM Issue #1490: Add an option to create a DPI protected library

Thanks for all the feedback.
> Wondering why you build a shared object instead of a static library (.a)?
Becaus... Todd Strader

09/01/2019

03:16 PM Issue #1494 (Resolved): verilator 4.018 build fails at 'make test' in examples/hello_world_c

Ahmed, thanks for the report.
Todd, thanks for pointing out the commit. Yes, part of that removal should have bee... Wilson Snyder

12:05 PM Issue #1494: verilator 4.018 build fails at 'make test' in examples/hello_world_c

Setting VERILATOR_ROOT to the root of your checkout will fix the problem.
Wilson, something about this changed at ... Todd Strader

03:18 AM Issue #1494 (Closed): verilator 4.018 build fails at 'make test' in examples/hello_world_c

When building verilator 4.018, the build failed when running make test in
examples/hello_world_c, with the followin... أحمد المحمودي

08/31/2019

12:12 AM Issue #1490: Add an option to create a DPI protected library

In addition to a test_regress, please add an example/, then refer to it in the bin/verilator docs.
Bunch of TODOs ... Wilson Snyder

08/30/2019

11:41 AM Issue #1490: Add an option to create a DPI protected library

To be clear, I think the MVP is feature complete (something self-contained that people could try out). There's a lot... Todd Strader

09:59 AM Issue #1490: Add an option to create a DPI protected library

Turns out that option #5 is to not need the DPI header at all. I'm not sure why I thought I needed it, but I don't.
... Todd Strader

08/29/2019

11:15 PM Issue #1435 (Closed): Report column numbers and source text in error messages

In 4.018.
Wilson Snyder

11:15 PM Issue #1487 (Closed): New WIDTH warnings on genvars

In 4.018.
Wilson Snyder

11:14 PM Issue #1491 (Closed): Add --dpi-hdr-only option

In 4.018.
Wilson Snyder

11:14 PM Issue #1475 (Closed): V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()

In 4.018.
Wilson Snyder

11:14 PM Issue #1442 (Closed): Enum value not made sized when enum sized

In 4.018.
Wilson Snyder

11:14 PM Issue #1429 (Closed): Feature request: elaboration tasks

In 4.018.
Wilson Snyder

11:14 PM Issue #1493 (Closed): Add XSim support to driver.pl

In 4.018.
Wilson Snyder

09:03 PM Issue #1493 (Resolved): Add XSim support to driver.pl

Great, thanks for the info. This is squashed and pushed now. Todd Strader

02:30 PM Issue #1493 (Feature): Add XSim support to driver.pl

These changes seem fine, you can squash and merge them as you feel appropriate.
When debugging a single test I som... Wilson Snyder

11:40 AM Issue #1493 (Closed): Add XSim support to driver.pl

Related to bug1490:
https://github.com/toddstrader/verilator-dev/tree/xsim
This is particularly useful to me beca... Todd Strader

11:14 PM Issue #1481 (Closed): Add rr support to the perl wrappers

In 4.018.
Wilson Snyder

11:14 PM Issue #1305 (Closed): Error messages do not contain hierarchical information

In 4.018.
Wilson Snyder

11:13 PM Verilator 4.018 Released

Verilator 4.018 2019-08-29
** When showing an error, show source code and offer suggestions of replacements.
... Wilson Snyder

10:36 PM Issue #1489 (Feature): Python support for Verilated designs

>the Python compile definition is VL_PYTHON. Is this ok? Because some variables have a VM_ prefix and some VL_.
VM... Wilson Snyder

08/28/2019

01:14 PM Issue #1483 (Closed): Make verilator_ext_tests head-to-head

Wilson Snyder

12:39 PM Issue #1483 (Resolved): Make verilator_ext_tests head-to-head

Done and done. Todd Strader

11:54 AM Issue #1483: Make verilator_ext_tests head-to-head

Ah, driver error. Feel free to squash and push. If you didn't already please contact the SweRV team to report their ... Wilson Snyder

09:59 AM Issue #1483: Make verilator_ext_tests head-to-head

That's what I've been doing. I just added ci/update_submodules.sh which does the remote submodule update for all sub... Todd Strader

12:34 PM Issue #1369: Raise error / warning on continous assignment to reg

Please use the unreleased git version, it intended to fix this.
Wilson Snyder

12:10 PM Issue #1369: Raise error / warning on continous assignment to reg

Hi,
It's been modified from 4.016. 4.016 version doesn't warn continuous assignments for the port (out) declared a... Kris Jeon

01:44 AM Issue #1491 (Resolved): Add --dpi-hdr-only option

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

12:07 AM Issue #1490: Add an option to create a DPI protected library

#3 is what I was thinking. Or, ideally refactor the DPI emit code to output something closer to pure Ast, and have V... Wilson Snyder

12:03 AM Issue #1490: Add an option to create a DPI protected library

> Why can't we build the dpi header in the same binary run? This seems cleaner for the user.
It would definitely b... Todd Strader

08/27/2019

11:07 PM Issue #1490 (Assigned): Add an option to create a DPI protected library

Will post comments about patch once get chance to review them.
>To build the library, I need to run Verilator agai... Wilson Snyder

11:02 PM Issue #1491 (Assigned): Add --dpi-hdr-only option

I'm fine with your squashing and pushing this.
However in the larger picture (perhaps worth discussing in the othe... Wilson Snyder

10:59 PM Issue #1483: Make verilator_ext_tests head-to-head

These changes look fine.
Personally I'd like to be able to run the tests pointing at a separate verilator tree (I ... Wilson Snyder

08:40 PM Issue #1369: Raise error / warning on continous assignment to reg

I wouldn't have thought your patch would be needed as the unreleased git version of verilator should have fixed this ... Wilson Snyder

08/24/2019

12:12 PM Issue #1491: Add --dpi-hdr-only option

I added a test for the new option. I believe this is ready to land now:
https://github.com/toddstrader/verilator-de... Todd Strader

08/23/2019

10:31 PM Issue #1491: Add --dpi-hdr-only option

It looks like I can just call compile() twice with different flags. I should be able to make something happen when I... Todd Strader

01:16 PM Issue #1491 (Closed): Add --dpi-hdr-only option

Towards bug1490. I have the feature coded, but am not sure about the best way to test this:
https://github.com/todd... Todd Strader

09:55 AM Issue #1490 (Closed): Add an option to create a DPI protected library

Jumping off from here:
https://www.veripool.org/boards/3/topics/3037
I've got a WIP branch to show where I'm goin... Todd Strader

08/22/2019

08:13 PM Issue #1483: Make verilator_ext_tests head-to-head

There's more to do here, but I'm focused on the DPI protected modules for now. But I believe this should be land-abl... Todd Strader

08/18/2019

01:09 AM Issue #1369: Raise error / warning on continous assignment to reg

In order to raise the warning for port, I've changed like the following:
In 'V3ParseGrammar.cpp'... Kris Jeon

08/14/2019

08:03 PM Issue #1489 (Feature): Python support for Verilated designs

I've improved the patchset by Patrick Stewart (proposed in bug1663) to add Python support for Verilated designs.
It ... Maarten De Braekeleer

08/12/2019

08:17 PM Issue #1487 (Resolved): New WIDTH warnings on genvars

Squashed, pushed and CI is green.
Verilator_ext_tests is working too. That CI environment is still a WIP, but is ... Todd Strader

07:26 PM Issue #1487: New WIDTH warnings on genvars

Great, tighter patch than I was expecting. Please squash and push.
& if have chance (if didn't already), check exten... Wilson Snyder

01:14 PM Issue #1487: New WIDTH warnings on genvars

Here is my proposed fix:
https://github.com/toddstrader/verilator-dev/tree/genvar-width
It appears the specific i... Todd Strader

08/10/2019

05:07 AM Issue #1487: New WIDTH warnings on genvars

Ok, the new test was written without looking at the old warning rules. My thinking was any const assignment to a unra... Wilson Snyder

08/09/2019

10:06 PM Issue #1487: New WIDTH warnings on genvars

Re: t_lint_width_genfor, I'm thinking the WIDTH warning in the procedural block "ri = i;" may not really be new. I w... Todd Strader

10:10 AM Issue #1487: New WIDTH warnings on genvars

Added two new tests in trunk, t_lint_width_genfor_bad which correctly fails
(though when this bug is fixed might hav... Wilson Snyder

08/08/2019

10:31 PM Issue #1276: vcd trace splits packed data type when it comes through a typedef

I'll try to pull the latest version and see what happens. It's been a while since I updated verilator. I'll update ... Christopher Russell

09:51 PM Issue #1468: Add Continuous Integration

P.S. Note that github just announced free open source CI via Actions. From what I can tell it's two active actions ea... Wilson Snyder

08:44 AM Issue #1486 (NoFixNeeded): Issue Linking on Mac OS

Glad you solved it.
Verilator uses the make-standard AR variable, so "make AR=foo ..."
Wilson Snyder

06:36 AM Issue #1486: Issue Linking on Mac OS

Wilson Snyder wrote:
> The _All files include some of the other files you already have listed, look at the contents ... Joseph Shaker

08/07/2019

09:23 PM Issue #1486: Issue Linking on Mac OS

The _All files include some of the other files you already have listed, look at the contents to determine which to in... Wilson Snyder

02:47 AM Issue #1486: Issue Linking on Mac OS

Wilson Snyder wrote:
> Maybe your "ar" is broken somehow? Try making a single "g++" command, where the file you spe... Joseph Shaker

05:18 AM Issue #1488: A strange code generated from a parametric module.

Thank you very much, Wilson! I was not aware about --pins-sc-uint switch! Slava B

08/06/2019

01:35 PM Issue #1488 (NoFixNeeded): A strange code generated from a parametric module.

uints are much faster so are the default. See the Connecting to SystemC part of the manual and the --pins-sc-uint/--... Wilson Snyder

11:51 AM Issue #1488 (NoFixNeeded): A strange code generated from a parametric module.

I've seen a strange code generated from a parametric module. I minimized this scenario to a few lines:... Slava B

08/05/2019

09:15 PM Issue #1363: CMake support

Ok, I'll hold off merging. If you could please merge from master into cmake, then merge cmake into your python branch... Wilson Snyder

06:37 PM Issue #1363: CMake support

I have applied your patch and think I have fixed the problem with t_flag_make_cmake_sc.pl
I would like to delay me... Maarten De Braekeleer

12:50 AM Issue #1363: CMake support

The sc example now works, thanks.
1. The attached has misc whitespace and other fixups you can please check over a... Wilson Snyder

02:35 AM Issue #1429 (Resolved): Feature request: elaboration tasks

Fixed in git towards 4.018.
Wilson Snyder