Project

General

Profile

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

Activity

From 07/01/2019 to 07/30/2019

07/30/2019

10:29 PM Issue #1363: CMake support
I agree users should do a "make" before "make install" however I think it's a feature if someone changes code to make... Wilson Snyder
01:08 PM Issue #1363: CMake support
* I did remove the dependency of `make install` on `all_nomsg` because if some verilator source is changed between th... Maarten De Braekeleer
01:03 AM Issue #1363: CMake support
Also please test that "make test" is clean with VERILATOR_AUTHOR_SITE=1 in your environment before you configure.
...
Wilson Snyder
12:57 AM Issue #1363: CMake support
Thanks, getting close.
I still can't "make examples" out of the box.
make examples
...
-- CMake ----...
Wilson Snyder
07:36 PM Issue #1484 (Duplicate): Parameter of an array of enumeration values breaks conditionals inside a...
Thanks for the report & test. This is a duplicate of bug1315, on which work is currently in progress.
Wilson Snyder
07:21 PM Issue #1484 (Duplicate): Parameter of an array of enumeration values breaks conditionals inside a...
This was tested using the Verilator 4.016 release. When using a parameter that is an array of enumeration values insi... John Martin

07/29/2019

03:58 PM Issue #1363: CMake support
I've added a systemc example. I've tested this using Accellera's systemc.
The test will only execute if systemc ha...
Maarten De Braekeleer

07/26/2019

11:31 PM Issue #1482 (Feature): Conditional event controls ("iff")
The event parsing needs some rewriting to handle this, but is relatively straight forward.
Added a disabled t_iff....
Wilson Snyder
12:14 AM Issue #1482 (Feature): Conditional event controls ("iff")
Section 9.4.2.3 of IEEE 1800-2017 allows "iff" qualifiers on @ event controls. The example code in 9.4.2.3 is fairly... Paul Donahue
10:59 PM Issue #1483 (Assigned): Make verilator_ext_tests head-to-head
I agree this is a good approach. Note Verilator, along with SwerRV is part of Chips Alliance, so if you want you can... Wilson Snyder
10:16 PM Issue #1483 (Closed): Make verilator_ext_tests head-to-head
This is still a WIP, but I think it warrants some discussion before I trudge ahead much further:
https://github.com/...
Todd Strader
06:51 PM Issue #1305 (Resolved): Error messages do not contain hierarchical information
Wilson Snyder
05:30 PM Issue #1305: Error messages do not contain hierarchical information
Sold. This has been squashed, pushed and is green. Todd Strader
04:30 PM Issue #1305: Error messages do not contain hierarchical information
Long term I think we need a nicer way to handle error messages but a project better left for later. So I agree #2 is... Wilson Snyder
03:51 PM Issue #1305: Error messages do not contain hierarchical information
I updated the branch with all these changes:
https://github.com/toddstrader/verilator-dev/tree/mod_stack_trace_10
...
Todd Strader
06:14 PM Issue #1363: CMake support
I've rewritten `configure.ac` to use a foreach loop.
I'll look into adding systemc support for the cmake script.
Maarten De Braekeleer
04:59 PM Issue #1363: CMake support
I pushed the contributors patch and typos patch - threw some other typo fixes in after searching for the same typo. T... Wilson Snyder
02:22 PM Issue #1363: CMake support
Hello folks
@Patrick Stewart
* I have modified the `make install` to copy the cmake scripts to /usr/share/verilat...
Maarten De Braekeleer
11:20 AM Issue #1363: CMake support
I'm using cmake 3.10.2. I'm running "make examples" inside the parent git repo checkout (not installed).
rm -...
Wilson Snyder
10:55 AM Issue #1363: CMake support
If you're putting it into share/verilator/cmake, you need to set the parent directory of ${CMAKE_CURRENT_LIST_DIR} on... Patrick Stewart
10:24 AM Issue #1481 (Resolved): Add rr support to the perl wrappers
Also, please add the bug # to the Changes, did this, e.g.
*** Add --rr, bug1481. [Todd Strader]
Thanks fo...
Wilson Snyder
10:23 AM Issue #1481: Add rr support to the perl wrappers
Number of stars is the importance. Usually big things 2, features are 3, bug fixes 4, but not a science.
Wilson Snyder
09:22 AM Issue #1481: Add rr support to the perl wrappers
Squashed, pushed and green.
Can you explain the number of stars for each entry in Changes? It appears that they a...
Todd Strader

07/25/2019

11:45 PM Issue #1481 (Feature): Add rr support to the perl wrappers
"gdb!" => \$opt_gdb,
+ "rr!" => \$opt_rr,
"gdbbt!" => \...
Wilson Snyder
11:28 PM Issue #1481 (Closed): Add rr support to the perl wrappers
As discussed in the forums:
https://www.veripool.org/boards/3/topics/3028
This allows users to record verilation ...
Todd Strader
10:50 PM Issue #1363: CMake support
Thanks for all the cleanups.
If you want just point me to a github branch and you can skip posting patch files.
...
Wilson Snyder
03:52 PM Issue #1363: CMake support
I've created a new pull request in which I add myself to docs/CONTRIBUTORS to certify that this and future contributi... Maarten De Braekeleer
03:34 PM Issue #1363: CMake support
Hello,
I think I have incorporated the comments in #5.
They can still be found at https://github.com/madebr/ver...
Maarten De Braekeleer
03:33 PM Issue #1363: CMake support
Hello Patrick,
Thanks for the comments!
I think I have incorporated your suggestions with some changes.
Maarten De Braekeleer
10:34 AM Issue #1363: CMake support
Thanks for taking this on Maarten, I haven't had time for a while.
I've just a had a quick go with it, when I instal...
Patrick Stewart
02:16 AM Issue #1363: CMake support
Nice job.
1. Please sumbit a one-time independent patch to edit docs/CONTRIBNUTORS to
add your name and certify t...
Wilson Snyder
10:39 PM Issue #1305: Error messages do not contain hierarchical information
Multiple cell visiting in AstParam is expected as it does the original cell, then the one it clones to de-parameteriz... Wilson Snyder
05:07 PM Issue #1305: Error messages do not contain hierarchical information
OK, none of the .out files are looking for internal faults now. And CI is green:
https://github.com/toddstrader/ver...
Todd Strader
03:27 PM Issue #1305: Error messages do not contain hierarchical information
Nope. Apparently during my git juggling of the _bad.out files, I missed that I committed two test which are now toss... Todd Strader
02:44 PM Issue #1305: Error messages do not contain hierarchical information
After the refactor in bug1315, I think we're probably/hopefully there:
https://github.com/toddstrader/verilator-dev/...
Todd Strader

07/24/2019

07:27 PM Issue #1363: CMake support
Hey
I've rebased the cmake patch and updated it a bit.
It would be nice if someone would review them.
The code...
Maarten De Braekeleer

07/23/2019

07:53 PM Issue #1315: Using an array in a function called from a parameter
Squashed, pushed and passing CI. Todd Strader
05:53 PM Issue #1315: Using an array in a function called from a parameter
>Do you think I should leave it as-is so that external users of the fetcher methods get a V3Number*? Or should I chan... Wilson Snyder
03:27 PM Issue #1315: Using an array in a function called from a parameter
> I would think that using an AstConst is ok also, as previously we would have fetched an V3Number with similar lifet... Todd Strader

07/22/2019

10:15 PM Issue #1479: Feature Request: VerilatedVcd callback on rolloverMB
Good work, in general it seems fine.
Please also have the patch add your name to docs/CONTRIBUTORS to acknowledge ...
Wilson Snyder
08:09 PM Issue #1479: Feature Request: VerilatedVcd callback on rolloverMB
I have rebased the initial patch on top of master, sorry for posting a second patch.
This patch adds a filename() ...
Marc Jessome
12:47 PM Issue #1475 (Assigned): V3Hashed.cpp Called isIdentical on non-hashed nodes, from Gate dedupe()
Got a test case which shows a replacement is going into a flop which itself is then subject to replacement. This is ... Wilson Snyder
02:34 AM Issue #1315: Using an array in a function called from a parameter
>I made variants of fetch* that return AstConst* for use inside SimulateVisitor and
>kept V3Number* methods for use ...
Wilson Snyder

07/19/2019

08:19 PM Issue #1315: Using an array in a function called from a parameter
Here is a candidate for #1:
https://github.com/toddstrader/verilator-dev/tree/simulate-w-nodes-2
Some things to d...
Todd Strader
03:36 PM Issue #1479: Feature Request: VerilatedVcd callback on rolloverMB
I'm not sure what problem you are forseeing. Perhaps give inheriting a try? We welcome patches that result. Wilson Snyder
03:06 PM Issue #1479: Feature Request: VerilatedVcd callback on rolloverMB
Thanks for the feedback Wilson.
I could definitely make this work by inheriting VerilatedVcd with access to filena...
Marc Jessome
11:57 AM Issue #1479 (Feature): Feature Request: VerilatedVcd callback on rolloverMB
I'm surprised there wasn't an accessor on filename() please add that regardless.
Do you really need a callback? Th...
Wilson Snyder

07/18/2019

05:13 PM Issue #1479 (Feature): Feature Request: VerilatedVcd callback on rolloverMB
I have the time to implement this feature, and am looking for input.
This is a feature for use in conjunction with...
Marc Jessome
12:02 PM Issue #1478 (Confirmed): -faligned-new does not work under Travis with clang
Wilson Snyder
11:41 AM Issue #1478 (Confirmed): -faligned-new does not work under Travis with clang
It was discovered in bug1468 that the -faligned-new flag causes errors when compiling Verilated code under Travis CI ... Todd Strader
12:01 PM Issue #1468 (Closed): Add Continuous Integration
Awesome work!
Wilson Snyder
11:45 AM Issue #1468: Add Continuous Integration
Squashed, pushed and the tests are green. Also for reference, bug1477 and bug1478 were opened as discussed above. Todd Strader
01:18 AM Issue #1468: Add Continuous Integration
Great, again. "more-travis" looks fine please squash merge it to trunk (or if get it to one commit just normal pull ... Wilson Snyder
12:29 AM Issue #1468: Add Continuous Integration
This is ready to push/discuss next steps:
https://github.com/toddstrader/verilator-dev/tree/more-travis
Here is t...
Todd Strader
10:32 AM Issue #1477 (Assigned): Add macOS coverage in Travis CI
Kevin Kiningham indicated he will look at this in about a month.
Wilson Snyder
09:56 AM Issue #1477 (Assigned): Add macOS coverage in Travis CI
.travis.yml has commented out lines for macOS testing. I was unable to get it to work and don't have a local macOS e... Todd Strader

07/15/2019

01:47 AM Issue #1435 (Resolved): Report column numbers and source text in error messages
Wilson Snyder
01:46 AM Issue #1435: Report column numbers and source text in error messages
Fixed in git towards 4.017. Also added suggestions, which are arguably even more useful.... Wilson Snyder
01:44 AM Issue #1439 (Closed): Add "included from" and macro expansion information in error messages
Macro expansion is very difficult as requires tracking recursive information on every token through the preprocessor.... Wilson Snyder

07/11/2019

04:16 PM Issue #1468: Add Continuous Integration
I was thinking that configure compiles but doesn't link, but it does seem to try a link, so I'm at a loss.
Wilson Snyder
03:55 PM Issue #1468: Add Continuous Integration
I don't know if it's the only clang problem, but there is a systemic problem when using clang under Travis which is c... Todd Strader
12:31 AM Issue #1439: Add "included from" and macro expansion information in error messages
Notes on how GCC tracks defines.
Debug can be seen with "g++ -fdebug-cpp -E -c file.cpp". line-map.c's linemap_du...
Wilson Snyder

07/10/2019

09:58 PM Issue #1305: Error messages do not contain hierarchical information
Reply in bug1315. Wilson Snyder
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... Todd Strader
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... Wilson Snyder
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...
Todd Strader

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. Shareef Jalloq
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 ... Wilson Snyder

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? Shareef Jalloq
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... Wilson Snyder
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... Shareef Jalloq

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.
Wilson Snyder
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.
Wilson Snyder
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...
Philipp Wagner
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...
Wilson Snyder
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 ... Todd Strader
05:48 PM Issue #1468: Add Continuous Integration
FYI I disabled t_leak.pl for now when under travis.
Wilson Snyder
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'... Wilson Snyder
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 ... Todd Strader
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...
Wilson Snyder

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

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

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. Patrick Collins
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...
Todd Strader
 

Also available in: Atom