Activity
From 08/11/2019 to 09/09/2019
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...
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...
Also available in: Atom