Activity
From 06/11/2019 to 07/10/2019
07/10/2019
- 09:58 PM Issue #1305: Error messages do not contain hierarchical information
- Reply in bug1315.
- 07:13 PM Issue #1305: Error messages do not contain hierarchical information
- So then would we have an AstNode* (or maybe AstConst*) in V3Number and set that pointer inside simulateVisitor::newNu...
- 09:58 PM Issue #1315: Using an array in a function called from a parameter
- The idea is every V3Number would point to the AstConst it's inside (excluding when inside the parser unless we fix th...
- 07:13 PM Issue #1315: Using an array in a function called from a parameter
- Wilson,
Some questions regarding #1:
* Just to make sure I understand, are you suggesting that user2/3p would g...
07/09/2019
- 11:21 AM Issue #1474: Document parse-order-dependency for config files
- Thanks, will give it a go when I have time. Cheers.
- 02:15 AM Issue #1474: Document parse-order-dependency for config files
- An absolute path wasn't really intended but isn't surprising since it was never thought about - would you be willing ...
07/08/2019
- 03:00 PM Issue #1474: Document parse-order-dependency for config files
- Creating the testcase shows me that you require an absolute path in the VLT file. Is that by design or a bug?
- 01:43 PM Issue #1474: Document parse-order-dependency for config files
- Yes, the .vlt should be first. If you don't see suppression working please reduce it to a test case and file a new b...
- 01:34 PM Issue #1474: Document parse-order-dependency for config files
- I've hit what I think is the same issue today but can't get the waivers to apply. I'm supplying the .wlt configurati...
07/07/2019
- 12:00 PM Issue #1476 (Duplicate): Please honour CFLAGS and CXXFLAGS provided by user
- Thanks, this was earlier reported via Gianfranco Costamagna and is applied in git.
- 12:49 AM Issue #1476 (Duplicate): Please honour CFLAGS and CXXFLAGS provided by user
- It was reported on Debian that verilator does not honor CFLAGS that are provided by user. The attached patch fixes th...
07/06/2019
- 10:11 PM Issue #1474 (Closed): Document parse-order-dependency for config files
- Perfect, thanks, pushed to git towards 4.018.
- 08:46 PM Issue #1474: Document parse-order-dependency for config files
- Hi Wilson,
Sorry for the slow response, looks like the email notification didn't reach me.
An updated patch wit... - 08:27 PM Issue #1442 (Resolved): Enum value not made sized when enum sized
- Fixed in git towards 4.018.
I tried a few alternatives to preserve the behavior I thought was best for WIDTH warni... - 07:27 PM Issue #1468: Add Continuous Integration
- Ok, cool. I'm pretty sure it's only happening under 14.04 in Travis (and only --vltmt). I tried to reproduce under ...
- 05:48 PM Issue #1468: Add Continuous Integration
- FYI I disabled t_leak.pl for now when under travis.
- 07:25 PM Issue #1305: Error messages do not contain hierarchical information
- Yes, I think if V3Simulate uses AstConst, then the only standalone number use left is in the parser and that shouldn'...
- 07:15 PM Issue #1305: Error messages do not contain hierarchical information
- I currently have it so that V3Numbers point back to their AstNode so at error time they can get location information ...
- 06:20 PM Issue #1305: Error messages do not contain hierarchical information
- I stumbled on the V3Hashed iterator problem earlier too, this is fixed in trunk.
I'm confused, why does AstNode ne...
07/04/2019
- 03:37 PM Issue #1468: Add Continuous Integration
- Feel free to push the badged README.pod. The more_travis branch currently seems to have hackage to debug t_leak so do...
07/03/2019
- 04:26 PM Issue #1468: Add Continuous Integration
- Couple more notes:
I'm pretty sure that t_leak is not flaky but just failing under Ubuntu 14.04. It probably look... - 11:53 AM Issue #1468: Add Continuous Integration
- Yeah, we only have three workers to play with regardless of OS, compiler, etc. I've set it up so that we can get the...
- 02:26 PM Issue #1305: Error messages do not contain hierarchical information
- OK, here's the latest iteration:
https://github.com/toddstrader/verilator-dev/tree/mod_stack_trace_8
I went ahead...
07/01/2019
- 11:19 PM Issue #1315: Using an array in a function called from a parameter
- Thanks! I'll try to poke at it and see how far I can get.
- 09:52 AM Issue #1468: Add Continuous Integration
- > t_leak seems to fail, can you debug?
Sure.
> Have the vltmt & vlt section only do a test_regress, not a test
G...
06/29/2019
- 09:31 PM Issue #1468: Add Continuous Integration
- 1. t_leak seems to fail, can you debug?
2. Have the vltmt & vlt section only do a test_regress, not a test, as thi... - 07:25 PM Issue #1468: Add Continuous Integration
- FYI got a false failure. I added automatic retry of failures (long overdue). Also turned down the verbosity.
- 12:26 PM Issue #1468: Add Continuous Integration
- FYI Travis caught I botched t_flag_csplit on old makes, fixed (I think).
- 10:40 AM Issue #1315: Using an array in a function called from a parameter
- Note bug1052 also is the same issue, so should be part of the test cases.
- 09:53 AM Issue #1315: Using an array in a function called from a parameter
- Patrick, would love to have you help on this. While certainly it will take a bit of time investment to understand th...
- 12:41 AM Issue #1315: Using an array in a function called from a parameter
- I just bumped into this too. Do you think this is a tractable problem for someone new to the Verilator codebase?
- 10:39 AM Issue #1052 (Duplicate): Localparam array can't be passed into parameter-called function
- bug1315 is the same issue, so tracking there.
- 10:24 AM Issue #385 (Closed): Dpi exported tasks with array inputs don't compile.
- Forgot to close; open array support was added in 3.918 2018-01-02.
- 10:16 AM Issue #1422 (NoFixNeeded): VPI hierarchy naming
- See last msg, think this will make naming compatible, if not please reopen.
06/28/2019
- 05:10 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- I've tried to reduce it along the lines you describe and I've failed. I'm skeptical of this apparoch because simply ...
- 04:45 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- That helps, but not enough to suggest what to fix so still think we need to get to a testcase. Basically, Verilator ...
- 10:33 AM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- Output w/patch:...
- 09:59 AM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- Presumably it's --debug that is hiding the issue as the code compiles differently.
Try --debugi 3. Also edit line... - 07:36 AM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- I tried with valgrind, the crash was reproducible, but it didn't report anything.
- 07:36 AM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- I'm afraid that I'm only seeing the crash without "--debug --gdbgt", I checked.
- 12:06 AM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- The logfile you attached showed no errors. Perhaps you're still in the passing bisect?
06/27/2019
- 11:34 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- Here's the last lines when running with "--debug --gdbgt":...
- 11:28 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- Is it possible to split that commit into a sequence of commits, purely for the purpose of bisecting it further or is ...
- 10:54 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- Bisecting was a good idea, unfortunately that's was a hyper large change, though I wouldn't have expected those edits...
- 10:42 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- I was able to run a bisection to find the first failing version of Verilator:...
- 10:21 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- This is the only other file in the design in addition to DressRehearsalTestBench.v
- 10:17 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- The parameterization happens in Chisel, so Chisel spits out unparameterized files, so the only parameterization I hav...
- 10:13 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- ...
- 10:04 PM Issue #1475 (AskedReporter): V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- There was recently a similar error resulting from parameters, are there parameters in DressRehersalTestBench.v?
Wh... - 06:21 PM Issue #1475: V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- This is with 4.016 as well as 4.014:
$ verilator --version
Verilator 4.014 2019-05-08 rev UNKNOWN_REV
- 06:19 PM Issue #1475 (Closed): V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
- I'm getting the error below in *BOLD* when I invoke Verilator. DressRehearsalTestBench.v is generated by Chisel: http...
- 10:00 PM Issue #1474: Document parse-order-dependency for config files
- Sorry you had to experiment to figure this out (I didn't realize myself ;) but major thanks for improving the docs.
... - 03:03 PM Issue #1474 (Closed): Document parse-order-dependency for config files
- When including a *.vlt configuration file with lint waivers, only the
global waivers (file == "*") are applied uncon... - 05:06 PM Issue #1468: Add Continuous Integration
- OK, the push works now and I see the authorization for Travis. I'm not sure how to tell Travis to start watching the...
- 04:38 PM Issue #1468: Add Continuous Integration
- Fixed a mis-setting, you should be able to push. If other problems might want to use git rather than http as a transp...
- 04:01 PM Issue #1468: Add Continuous Integration
- OK, I believe everything is push-able now:
https://github.com/toddstrader/verilator-dev/tree/travis
https://travis-...
06/26/2019
- 11:57 PM Issue #1473: How to get no `line pragmas in preprocessor
- Wilson Snyder wrote:
> Please see the -P option in the manual.
>
> If you're writing a parser, you might want to ... - 10:42 PM Issue #1473 (NoFixNeeded): How to get no `line pragmas in preprocessor
- Please see the -P option in the manual.
If you're writing a parser, you might want to consider handling `line for ... - 08:47 PM Issue #1473 (NoFixNeeded): How to get no `line pragmas in preprocessor
- As I understand, Verilator preprocessor inserts `line pragmas instead of `include and other directives.
It can appea... - 11:09 PM Issue #1468: Add Continuous Integration
- Sorry, what I meant for t_gantt was just pass --threads 2 always then fix the test. I just pushed that change.
I ... - 10:12 PM Issue #1468: Add Continuous Integration
- >> On my travis-ci.com dashboard there is a button to request access for GitHub organizations I'm in. Once we land th...
- 10:50 PM Issue #1472 (AskedReporter): Memory not updating in for loop
- Please attach a complete standalone example so I can try it, thanks. Ideally this would be in test_regress format as...
- 12:12 PM Issue #1472 (NotEnoughInfo): Memory not updating in for loop
- We are facing an issue with following verilog code
Snippets of the relevant portion of code is shown below :
--...
06/25/2019
- 01:33 PM Issue #1468: Add Continuous Integration
- UNOPTTHREADS is a bit finicky, it's based on the parallelism it can find
divided by threads. Can you see if we can ... - 12:21 PM Issue #1468: Add Continuous Integration
- As advertised, I am now skipping these failing vltmt tests if the core count is < 3.
I'm guessing that the large n...
06/24/2019
- 12:17 PM Issue #1468: Add Continuous Integration
- OK, so I'm not sure what to do with --vltml. Based on reading the man page and this issue:
https://www.veripool.org...
06/22/2019
- 09:04 PM Issue #1422: VPI hierarchy naming
- Stumbled into some other code related to handling TOP, note the existing code allows passing "" when constructing a m...
- 09:01 PM Issue #1439: Add "included from" and macro expansion information in error messages
- Includes easier than thought as preproc already records the include tree. Also added easy insertion point for adding ...
- 05:01 PM Issue #1439: Add "included from" and macro expansion information in error messages
- I haven't done anything yet. Please feel free.
- 04:52 PM Issue #1439: Add "included from" and macro expansion information in error messages
- Have you looked at this?, if not will look at the include part first which seems easier.
- 04:50 PM Issue #1462: signal redeclaration is not reported
- Thanks for checking, was missing many other cases, and also warning on duplicated ports.
Fixed in git towards 4.01...
06/21/2019
- 10:01 AM Issue #1468: Add Continuous Integration
- I guess I just wasn't patient enough. I was looking for the cache to be shared between the test slices (I had one ru...
06/20/2019
- 10:18 PM Issue #1468: Add Continuous Integration
- Woah. I assumed that you couldn't share state between instances because:
https://docs.travis-ci.com/user/build-stag... - 12:39 PM Issue #1468: Add Continuous Integration
- Hi,
I just wanted to point out our ongoing activities of LibreCores CI. It is a jenkins-based CI system we are con... - 02:07 PM Issue #1470: VPI systemtf
- Ah, you are absolutely right. Good, that I didn't start trying to implement anything so far :)
I think we should b... - 02:04 PM Issue #1470: VPI systemtf
- Adding a way to overload $display would be ok but you'd have a larger problem that display takes arguments and the DP...
- 01:44 PM Issue #1470: VPI systemtf
- How would you fancy an implementation that only allows to overwrite systemtf for a limited set of systemtf (in partic...
- 01:41 PM Issue #1470: VPI systemtf
- It is actually even worse: It only uses the systemtf to re-direct $display-style system functions to its output.
A... - 01:33 PM Issue #1470 (Feature): VPI systemtf
- Note Verilator does support a language extension where arbitrary doller-user functions can instead be converted into ...
- 12:58 PM Issue #1470 (Feature): VPI systemtf
- Hi,
VPI allows to add and overwrite System Tasks and Functions. I propose to add the infrastructure to allow for t... - 01:48 PM Issue #1469: VPI module
- Thanks for the background. I agree adding the type to VpiScope seems most preferred.
- 01:38 PM Issue #1469: VPI module
- Oh, I forgot to add: Major reason for this proposal was that nothing outside a VerilatedScope can be accessed anyways...
- 01:34 PM Issue #1469: VPI module
- Essentially, cocotb accesses the signals only by their name. It only uses the vpiModule iterator to find toplevel mod...
- 01:26 PM Issue #1469 (Feature): VPI module
- My first reading of this was you needed module scopes, but that is already there, so I think you're saying you want s...
- 12:58 PM Issue #1469: VPI module
- But we don't want to limit the iteration to just modules, correct? I'm sure I still don't understand enough about ho...
- 12:51 PM Issue #1469 (Closed): VPI module
- Hi,
with the ongoing integration of Verilator with cocotb, one of the first steps is to get a minimal working VPI ... - 11:36 AM Issue #1305: Error messages do not contain hierarchical information
- Oh, ack. That is part of what VL_LEAK_CHECK/V3Broken::isAllocated is for, right? I clearly hadn't gotten around to ...
06/19/2019
- 08:58 PM Issue #1305: Error messages do not contain hierarchical information
- Clarification on AstNode clean up:
Will AstNodes ever be destroyed without calling deleteNode()? Or is that always ... - 12:45 PM Issue #1468: Add Continuous Integration
- I would hope that the ccache is persistent across their farm, or at least it should, otherwise I don't see why they w...
06/18/2019
- 10:19 PM Issue #1468: Add Continuous Integration
- > https://github.com/verilator/verilator can be now used instead of git.veripool.org/git/verilator, they are identica...
- 05:47 PM Issue #1468 (Feature): Add Continuous Integration
- https://github.com/verilator/verilator can be now used instead of git.veripool.org/git/verilator, they are identical....
- 02:55 PM Issue #1468: Add Continuous Integration
- Aaaand, I just noticed:
https://github.com/verilator/verilator_ext_tests
That's pretty timely. I had originally ...
06/17/2019
- 10:38 PM Issue #1468 (Closed): Add Continuous Integration
- This builds Verilator and runs the entire test suite within Travis-CI:
https://github.com/toddstrader/verilator-dev/... - 10:11 AM Issue #1462: signal redeclaration is not reported
- I tried the first example with version 4.016 but it did not worked. Verilator did not raise warning or error on dupli...
06/16/2019
- 01:59 PM Issue #1467 (Closed): Constant function evaluator incorrect on return of function var
- In 4.016.
- 01:59 PM Issue #1465 (Closed): --savable generates invalid c++ for some packed arrays
- In 4.016.
- 01:59 PM Issue #1462 (Closed): signal redeclaration is not reported
- In 4.016.
- 01:59 PM Issue #1460 (Closed): fstapi.c compile errors.
- In 4.016.
- 01:59 PM Issue #1458 (Closed): Dotted references to type parameters do not have the correct size
- In 4.016.
- 01:59 PM Issue #1456 (Closed): sameHash error on type parameters after 4.014
- In 4.016.
- 01:59 PM Issue #1427 (Closed): Incorrect real parameter assignment
- In 4.016.
- 01:59 PM Issue #1455 (Closed): Multi-threaded verilated simulation on single-core
- In 4.016.
- 01:59 PM Issue #1444 (Closed): Invalid xml output generated when code contains functions with string argum...
- In 4.016.
- 01:58 PM Issue #1441 (Closed): incorrect handling of strings in display and sformatf
- In 4.016.
- 01:58 PM Issue #1443 (Closed): Calling $display with real %t argument causes segfault in _vl_vsformat
- In 4.016.
- 01:58 PM Issue #1437 (Closed): -Wsign-compare warning in verilated.cpp
- In 4.016.
- 01:58 PM Issue #1449 (Closed): deferred assertion support
- In 4.016.
- 01:58 PM Issue #1433 (Closed): Warnings from unused parameters in verilated_fst_c.h
- In 4.016.
- 01:58 PM Issue #1436 (Closed): Add an option to silence the command-failed message
- In 4.016.
06/15/2019
- 01:38 PM Issue #1414 (Duplicate): Multi dimensional variable in function causing issues
- Sorry, forgot to reply to this when filed. This is duplicate of bug1315, basically at present you can't have unpacked...
- 01:35 PM Issue #1429 (Feature): Feature request: elaboration tasks
- Did some poking on this, will add it to the soon-to-work on list. If you or someone has time to create a patch it wo...
- 12:10 PM Issue #1467 (Resolved): Constant function evaluator incorrect on return of function var
- Thanks for the good test case.
Fixed in git towards 4.015.
- 02:56 AM Issue #1467 (Confirmed): Constant function evaluator incorrect on return of function var
- Well, this one is entertaining. Basically this:
return index_calc;
The function is already using index_ca... - 12:11 AM Issue #1467 (Closed): Constant function evaluator incorrect on return of function var
- Trying to debug the root cause of multiple errors. Here is a minimal example I've managed to extract:...
06/14/2019
- 11:17 PM Issue #364: blocking & non-blocking assigns -- verilator issues error when no logical conflict ex...
- I wonder if some simple fix can be invented here, like splitting array into 2 arrays internally, one modeling a regis...
- 10:42 PM Issue #1465 (Resolved): --savable generates invalid c++ for some packed arrays
- Thanks much.
Pushed to git towards 4.015.
- 01:39 PM Issue #1465: --savable generates invalid c++ for some packed arrays
- Changes made, please find the new patch attached.
- 12:44 PM Issue #1465: --savable generates invalid c++ for some packed arrays
- Strings can't be packed arrays, so we're probably ok there. If you're ambitious try adding an array of strings to th...
- 11:34 AM Issue #1465: --savable generates invalid c++ for some packed arrays
- I see yes, I never bothered to look at the definition of @isWide@ etc, I was just mirroring the code in @emitVarDecl@...
- 11:11 AM Issue #1465 (AskedReporter): --savable generates invalid c++ for some packed arrays
- First thanks for making a patch and improving the tests. The patch seems reasonably close, just a few little things....
- 08:58 AM Issue #1465 (Closed): --savable generates invalid c++ for some packed arrays
- When trying to use @--savable@ I noticed compiler errors in the generated C++ code for the save/restore routines. Thi...
06/12/2019
- 11:17 PM Issue #1462 (Resolved): signal redeclaration is not reported
- Yes, some duplicates weren't reported.
Fixed in git towards 4.015.
- 01:38 PM Issue #1462 (Closed): signal redeclaration is not reported
- I am using verilator to lint verilog modules which will be then synthesized by Xilinx tools. Prior to synthesis veril...
- 01:38 AM Issue #1460 (Resolved): fstapi.c compile errors.
- Looks good, thanks for the efforts.
Pushed to git towards 4.015.
- 12:47 AM Issue #1460: fstapi.c compile errors.
- Hi,
Yes, did it. Here's the new and improved patch that hopefully has a minimal effect on everyone else. I added...
06/11/2019
- 11:00 PM Issue #1458 (Resolved): Dotted references to type parameters do not have the correct size
- 11:00 PM Issue #1458: Dotted references to type parameters do not have the correct size
- Golden, thanks. Looks like one of those sort of patches which requires hours of thought to result in just one patche...
- 11:01 AM Issue #1458: Dotted references to type parameters do not have the correct size
- OK, so I actually ran all the tests this time:
https://github.com/toddstrader/verilator-dev/tree/samehash-test
Th... - 10:52 PM Issue #1460 (AskedReporter): fstapi.c compile errors.
- Thanks for debugging.
_WORDSIZE is defined in sys/types.h and your patch moves the test above this include, so I t... - 06:20 AM Issue #1460 (Closed): fstapi.c compile errors.
- Hi,
I get many errors like the following with Verilator 4.014 (and before) when compiling fstapi using mingw32 (un...
Also available in: Atom