Project

General

Profile

[logo] 
 
Home
News
Activity
About/Contact
Major Tools
  Dinotrace
  Verilator
  Verilog-mode
  Verilog-Perl
Other Tools
  BugVise
  CovVise
  Force-Gate-Sim
  Gspice
  IPC::Locker
  Rsvn
  SVN::S4
  Voneline
  WFH
General Info
  Papers

Activity

From 08/23/2019 to 09/21/2019

09/20/2019

03:40 PM Issue #1470: VPI systemtf
Wilson Snyder wrote:
> Perhaps when suppressed this should be setting the error return code so vpi_chk_error returns...
Stefan Wallentowitz
03:32 PM Issue #1470: VPI systemtf
Perhaps when suppressed this should be setting the error return code so vpi_chk_error returns something sane e.g. lev... Wilson Snyder
02:30 PM Issue #1470: VPI systemtf
Okay, here is one that is equally hacky and elegant on both sides. Instead of making the symbol weak, how about just ... Stefan Wallentowitz
04:25 AM Issue #1470: VPI systemtf
Thanks, Wilson, I was already thinking about the issue with other compilers but only came up with LLVM :) I think it ... Stefan Wallentowitz
01:47 PM Issue #1512: Make callCbs public (again) and return if callbacks were called
Hi,
conceptually the VPI library is generic in cocotb and it used some init hooks to install the callbacks. But Ve...
Stefan Wallentowitz
01:30 PM Issue #1490: Add an option to create a DPI protected library
More questions about the in-AST refactor:
Currently "--dpi-protect secret" creates foo.sv and foo.cpp. It also sl...
Todd Strader
05:01 AM Issue #1511: Add --public-flat-rw flag
Thanks for your insights, I am a bit torn apart on this one. But I believe overall the usability wins in this case. T... Stefan Wallentowitz

09/19/2019

11:01 PM Issue #1512 (Assigned): Make callCbs public (again) and return if callbacks were called
I'm ok applying this conceptually, but do you think this is the API you really want to use instead of abstracting thr... Wilson Snyder
08:17 PM Issue #1512 (Assigned): Make callCbs public (again) and return if callbacks were called
For the cocotb support we need VPI callbacks of all sorts. The access to callCbs() was public before but has not been... Stefan Wallentowitz
10:52 PM Issue #1511 (Assigned): Add --public-flat-rw flag
I really don't like expanding the userfullness use of public. There's two problems:
1. Users will probably set it...
Wilson Snyder
05:16 AM Issue #1511: Add --public-flat-rw flag
This is a compromise between --public (deprecated) and requiring the designer to add the comments. By restricting it ... Stefan Wallentowitz
05:04 AM Issue #1511 (Assigned): Add --public-flat-rw flag
Hi,
the purpose of these patches is to add a new switch --public-flat-rw that changes the behavior as if /*verilat...
Stefan Wallentowitz
10:21 PM Issue #1470: VPI systemtf
Weak seems somewhat hacky, but I can live with hacky. You do however need to do not break non-GCC compilers e.g. MSV... Wilson Snyder
09:00 PM Issue #1470: VPI systemtf
Hi,
sorry, took much longer as I hoped to even look into it. Attached please find a first, rather trivial patch, t...
Stefan Wallentowitz

09/18/2019

04:24 PM Issue #1510 (Resolved): VPI: Get vpiType string
Great. Pushed to git for eventual release 4.020.
Wilson Snyder
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. Stefan Wallentowitz
11:19 AM Issue #1510: VPI: Get vpiType string
Yes, I am at it right now. Sorry, slipped me :) Stefan Wallentowitz
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... Wilson Snyder
11:09 AM Issue #1510 (Resolved): VPI: Get vpiType string
vpi_get_str: Get vpiType string Stefan Wallentowitz
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.
Wilson Snyder
11:14 AM Issue #1509: Fix variable cast in VPI
Updated patch for current HEAD Stefan Wallentowitz
10:00 AM Issue #1509 (Resolved): Fix variable cast in VPI
Fix cast from Var to base type when getting handle in VPI Stefan Wallentowitz

09/17/2019

08:09 PM Issue #1507: Linting error: backward indexing
It worked fine. Thanks!
Hao Shi
07:17 PM Issue #1507: Linting error: backward indexing
Sorry, not having it in master was my fault, try again.
Wilson Snyder
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,...
Hao Shi
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. Wilson Snyder
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...
Hao Shi
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. Wilson Snyder
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...
Hao Shi
01:18 AM Issue #1507 (Resolved): Linting error: backward indexing
Thanks for the report, simple enough to fix.
Fixed in git towards 4.420.
Wilson Snyder
12:58 AM Issue #1507 (Resolved): 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...
Hao Shi
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:... Todd Strader
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... Wilson Snyder
01:13 PM Issue #1508 (NoFixNeeded): CI is failing on CPAN installs
Travis jobs are failing while installing CPAN modules:... Todd Strader

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.... Wilson Snyder
07:15 PM Issue #1506 (NoFixNeeded): Compile with uvm-systemc
Is there any way or example to compile the verilator with uvm-systemc? SKY LAI

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...
Wilson Snyder
10:24 PM Issue #1505: [Bug] cannot understand multi-dimensional array interfaces
Verilog-Perl fixed in version 3.468.
Wilson Snyder
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.
Wilson Snyder
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 ... Ânderson Ignacio Da Silva

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... Rohit Yadav
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 ... Wilson Snyder
02:17 PM Issue #1504 (Closed): Produce man files during 'make all'
./nodist/install_test runs cleanly:... Todd Strader
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...
Wilson Snyder
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... Todd Strader
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 .... Wilson Snyder

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 ... Todd Strader
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:... Todd Strader
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...
Rohit Yadav

09/06/2019

05:21 PM Issue #1502 (Closed): -LDLIBS isn't an option
(This can go to closed immediately as trivial doc bug.)
Wilson Snyder
05:06 PM Issue #1502 (Resolved): -LDLIBS isn't an option
Done. Todd Strader
05:00 PM Issue #1502 (Assigned): -LDLIBS isn't an option
Huh. Yes, guess de-document it. Feel free to squash & push.
Wilson Snyder
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... Todd Strader
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...
Wilson Snyder
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. ... Dan Petrisko

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... Todd Strader
10:24 AM Issue #1499 (Resolved): Have V3EmitC::emitcSyms honor --output-split-cfuncs
Done. This has been pushed to git now. Todd Strader
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.
Wilson Snyder
01:29 AM Issue #1496 (Resolved): Support additional file I/O functions $fseek, $ftell, $frewind
Thanks again.
Pushed to git towards 4.020.
Wilson Snyder

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 (Resolved): 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 (Resolved): 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 (Resolved): 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 (Assigned): 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
 

Also available in: Atom