Activity
From 08/05/2019 to 09/03/2019
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...
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'...
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 ...
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 ... - 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... - 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...
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...
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...
- 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...
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 ...
- 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...
- 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 ..."
- 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 ...
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...
- 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... - 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!
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/--...
- 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:...
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...
- 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... - 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... - 02:35 AM Issue #1429 (Resolved): Feature request: elaboration tasks
- Fixed in git towards 4.018.
Also available in: Atom