Project

General

Profile

[logo] 
 
Home
About/Contact
Major Tools
  Dinotrace
  Verilator
  Verilog-mode
  Verilog-Perl
Other Tools
  IPC::Locker
  Parallel::Forker
  Voneline
General Info
  Papers

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... 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
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
 

Also available in: Atom