Activity
From 08/20/2019 to 09/18/2019
09/18/2019
- 04:24 PM Issue #1510 (Resolved): VPI: Get vpiType string
- Great. Pushed to git for eventual release 4.020.
- 11:45 AM Issue #1510: VPI: Get vpiType string
- Patch updated with test. Actually I found vpi_var the better test as it tests vpi_get_str too.
- 11:19 AM Issue #1510: VPI: Get vpiType string
- Yes, I am at it right now. Sorry, slipped me :)
- 11:18 AM Issue #1510 (Assigned): VPI: Get vpiType string
- Thanks for the patch, makes sense, could you also please add a test to t_vpi_get.cpp or one of the other tests? If n...
- 11:09 AM Issue #1510 (Closed): VPI: Get vpiType string
- vpi_get_str: Get vpiType string
- 11:25 AM Issue #1509 (Resolved): Fix variable cast in VPI
- Thanks for the good patch. Applied & pushed to git for eventual 4.016 release.
- 11:14 AM Issue #1509: Fix variable cast in VPI
- Updated patch for current HEAD
- 10:00 AM Issue #1509 (Closed): Fix variable cast in VPI
- Fix cast from Var to base type when getting handle in VPI
09/17/2019
- 08:09 PM Issue #1507: Linting error: backward indexing
- It worked fine. Thanks!
- 07:17 PM Issue #1507: Linting error: backward indexing
- Sorry, not having it in master was my fault, try again.
- 06:55 PM Issue #1507: Linting error: backward indexing
- Let me know if it is better to open a new issue, since it is unrelated topic.
I am on the master branch right now,... - 06:15 PM Issue #1507: Linting error: backward indexing
- It's not released (until next monthish), use master branch, that is don't use any tag.
- 06:02 PM Issue #1507: Linting error: backward indexing
- Wilson Snyder wrote:
> You can get it now from git, see the Install section. Typically releases (tarballs) are ever... - 05:25 PM Issue #1507: Linting error: backward indexing
- You can get it now from git, see the Install section. Typically releases (tarballs) are every month or so.
- 04:45 PM Issue #1507: Linting error: backward indexing
- Wilson Snyder wrote:
> Thanks for the report, simple enough to fix.
>
> Fixed in git towards 4.420.
Thanks for... - 01:18 AM Issue #1507 (Resolved): Linting error: backward indexing
- Thanks for the report, simple enough to fix.
Fixed in git towards 4.420.
- 12:58 AM Issue #1507 (Closed): Linting error: backward indexing
- I think I met a false error in linting:
Slice selection '[2:2]' has backward indexing versus data type's '[0:3]'
Sl... - 02:53 PM Issue #1508 (NoFixNeeded): CI is failing on CPAN installs
- OK, great. I didn't look closely enough at the log before, but now I see:...
- 01:28 PM Issue #1508: CI is failing on CPAN installs
- Note CPAN listed Parallel::Forker but it didn't seem to find it in Travis. This morning I tried pushing a new Parall...
- 01:13 PM Issue #1508 (NoFixNeeded): CI is failing on CPAN installs
- Travis jobs are failing while installing CPAN modules:...
09/15/2019
- 09:09 PM Issue #1506 (NoFixNeeded): Compile with uvm-systemc
- UVM-SystemC as I understand it is just another library which should not care about regular SystemC verilated modules....
- 07:15 PM Issue #1506 (NoFixNeeded): Compile with uvm-systemc
- Is there any way or example to compile the verilator with uvm-systemc?
09/12/2019
- 11:14 PM Issue #1505 (WillNotFix): [Bug] cannot understand multi-dimensional array interfaces
- Thanks for the report.
I created a test case and this is not supported in the two commercial simulators I could ch... - 10:24 PM Issue #1505: [Bug] cannot understand multi-dimensional array interfaces
- Verilog-Perl fixed in version 3.468.
- 02:45 PM Issue #1505 (Confirmed): [Bug] cannot understand multi-dimensional array interfaces
- Agreed this is a mistake in translation of the IEEE grammar. Also applies to Verilog-Perl which will get fixed first.
- 02:06 PM Issue #1505 (WillNotFix): [Bug] cannot understand multi-dimensional array interfaces
- Verilator 4.018 cannot parse understand multi-dimensional array using interfaces structures from system verilog. The ...
09/11/2019
- 07:11 PM Issue #1503: [VerilogAMS]: Unknown Language Specified
- Thanks for the response. I apologize, since it was mentioned in the options, I thought may be it is supported, althou...
- 11:19 AM Issue #1503 (WillNotFix): [VerilogAMS]: Unknown Language Specified
- Verilator supports almost no AMS, basically wreal and one or two functions, and this is unlikely to be improved even ...
- 02:17 PM Issue #1504 (Closed): Produce man files during 'make all'
- ./nodist/install_test runs cleanly:...
- 12:59 PM Issue #1504: Produce man files during 'make all'
- To test installation run ./nodist/install_test
This isn't part of the regression as builds differently.
If that... - 12:01 PM Issue #1504: Produce man files during 'make all'
- Yeah, on that note I didn't have latex installed until just now. I tried to 'make dist' but failed to make README.pd...
- 11:22 AM Issue #1504: Produce man files during 'make all'
- I think the original thinking was most people won't want to install tex. They do have to have perl. So I'd suggest ....
09/10/2019
- 12:31 PM Issue #1504: Produce man files during 'make all'
- t_dist_manifest is failing now because the .1 files are both in the DISTFILES list and in MANIFEST.SKIP. I was just ...
- 11:06 AM Issue #1504 (Closed): Produce man files during 'make all'
- It appears the intention is to build the man files when building the 'all' target:...
- 01:55 AM Issue #1503 (WillNotFix): [VerilogAMS]: Unknown Language Specified
- Hi,
I am new to Verilator. I am interested in simulating several verilogams test cases with Verilator. I ran the f...
09/06/2019
- 05:21 PM Issue #1502 (Closed): -LDLIBS isn't an option
- (This can go to closed immediately as trivial doc bug.)
- 05:06 PM Issue #1502 (Resolved): -LDLIBS isn't an option
- Done.
- 05:00 PM Issue #1502 (Assigned): -LDLIBS isn't an option
- Huh. Yes, guess de-document it. Feel free to squash & push.
- 04:58 PM Issue #1502 (Closed): -LDLIBS isn't an option
- The --help mentions -LDLIBS but that isn't actually an option. Perhaps there was a thought about plumbing both LDFLA...
- 11:10 AM Issue #1501 (Confirmed): Support bind statements within generate blocks
- Thanks for the test.
This is a bug due to Bind being expanded up front before parameters are resolved.
Note t... - 06:28 AM Issue #1501 (Confirmed): Support bind statements within generate blocks
- A bind statement within a generate if, if the generate if evaluates to false, should not bind. ...
09/04/2019
- 01:55 PM Issue #1490: Add an option to create a DPI protected library
- More for my own notes than anything else: running XSim with a static DPI library is not as simple as I had earlier in...
- 10:24 AM Issue #1499 (Resolved): Have V3EmitC::emitcSyms honor --output-split-cfuncs
- Done. This has been pushed to git now.
- 01:33 AM Issue #1499 (Assigned): Have V3EmitC::emitcSyms honor --output-split-cfuncs
- Good stuff, thanks.
Once you update tests feel free to squash and push.
- 01:29 AM Issue #1496 (Resolved): Support additional file I/O functions $fseek, $ftell, $frewind
- Thanks again.
Pushed to git towards 4.020.
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...
- 08:33 PM Issue #1499: Have V3EmitC::emitcSyms honor --output-split-cfuncs
- Proposed change:
https://github.com/toddstrader/verilator-dev/tree/sym-split - 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...
- 12:42 PM Issue #1496: Support additional file I/O functions $fseek, $ftell, $frewind
- I agree on the certificate and ack the request explicitly.
- 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... - 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... - 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 ... - 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. ... - 11:55 AM Issue #1369: Raise error / warning on continous assignment to reg
- See https://www.veripool.org/projects/verilator/wiki/Installing
- 11:23 AM Issue #1369: Raise error / warning on continous assignment to reg
- Oh, where can I get the unreleased git version ?
- 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 ... - 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...
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... - 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 ... - 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 ...
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...
- 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.
...
08/29/2019
- 11:15 PM Issue #1435 (Closed): Report column numbers and source text in error messages
- In 4.018.
- 11:15 PM Issue #1487 (Closed): New WIDTH warnings on genvars
- In 4.018.
- 11:14 PM Issue #1491 (Closed): Add --dpi-hdr-only option
- In 4.018.
- 11:14 PM Issue #1475 (Closed): V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- In 4.018.
- 11:14 PM Issue #1442 (Closed): Enum value not made sized when enum sized
- In 4.018.
- 11:14 PM Issue #1429 (Closed): Feature request: elaboration tasks
- In 4.018.
- 11:14 PM Issue #1493 (Closed): Add XSim support to driver.pl
- In 4.018.
- 09:03 PM Issue #1493 (Resolved): Add XSim support to driver.pl
- Great, thanks for the info. This is squashed and pushed now.
- 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... - 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... - 11:14 PM Issue #1481 (Closed): Add rr support to the perl wrappers
- In 4.018.
- 11:14 PM Issue #1305 (Closed): Error messages do not contain hierarchical information
- In 4.018.
- 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.
... - 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...
08/28/2019
- 01:14 PM Issue #1483 (Closed): Make verilator_ext_tests head-to-head
- 12:39 PM Issue #1483 (Resolved): Make verilator_ext_tests head-to-head
- Done and done.
- 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 ...
- 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...
- 12:34 PM Issue #1369: Raise error / warning on continous assignment to reg
- Please use the unreleased git version, it intended to fix this.
- 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... - 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.
- 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...
- 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...
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... - 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... - 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 ... - 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 ...
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...
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...
- 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... - 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...
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...
Also available in: Atom