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 01/26/2016 to 02/24/2016

02/24/2016

10:04 PM Issue #1041 (NoFixNeeded): Verilator not running on Ubuntu 14.04 (due to use of static object)
I am able to run verilator fine on scientific linux but on Ubuntu 14.04 the compiled object file fails at a segmentat... Yura Pyatnychko
02:42 PM Issue #1036: Reset fails to respond when driven from vector containing clock enable
This looks very similar to a problem we had with vectors of clocks:
http://www.veripool.org/issues/1009-Verilator-...
Todd Strader
02:22 AM Issue #1036 (Confirmed): Reset fails to respond when driven from vector containing clock enable
Attributes such as the clock enables and clocks themselves are propagated signal-by-signal rather than bit-of-signal.... Wilson Snyder

02/22/2016

07:15 PM Issue #1040: DPI string parameter and readmemh
Yes, I will give it a try. Thanks for the hint! Stefan Wallentowitz
07:12 PM Issue #1040: DPI string parameter and readmemh
First time this has probably come up. Can you try making a patch to fix this? Add a "extern void VL_READMEM_N()" fun... Wilson Snyder
07:05 PM Issue #1040 (Closed): DPI string parameter and readmemh
Hi,
I am using this construct:...
Stefan Wallentowitz
02:02 PM Issue #1034: Extend V3Dead to be able to remove empty package modules/cells
Thanks. I'll provide an update.
We've been using an extended version of this patch internally that also removes un...
Johan Bjork
04:17 AM Issue #1034: Extend V3Dead to be able to remove empty package modules/cells
Seems reasonable. Only obvious think I see missing is that when a scope is deleted its aboveScope's user1 should be d... Wilson Snyder
01:46 PM Issue #1037: Verilator truncates statement incorrectly for pattern assignments
thanks! Johan Bjork
04:01 AM Issue #1037 (Resolved): Verilator truncates statement incorrectly for pattern assignments
Fixed in git towards 3.881.
Wilson Snyder
01:42 PM Issue #1035: Constructors and change detection does not honor --output-split-cfuncs
I'll provide a fix. I hacked it up internally and it works well for us, but I'll have to put in some work to make it ... Johan Bjork
03:34 AM Issue #1035 (AskedReporter): Constructors and change detection does not honor --output-split-cfuncs
Would you like to provide a fix, or do you think this is no longer that relevant?
Wilson Snyder

02/19/2016

05:30 AM Issue #1037 (Closed): Verilator truncates statement incorrectly for pattern assignments
... Johan Bjork

02/16/2016

03:56 PM Issue #1035: Constructors and change detection does not honor --output-split-cfuncs
Turns out the massive change detection function is partly due to some ignored (at the moment) UNOPTFLAT warnings as w... Johan Bjork

02/14/2016

05:37 PM Issue #1036: Reset fails to respond when driven from vector containing clock enable
There's a formatting problem in the commands to reproduce the issue. There should be two commands (separated by a ne... Neil Turton
03:12 PM Issue #1036 (Confirmed): Reset fails to respond when driven from vector containing clock enable
In the attached test, the active low reset output fails to go high when the active high input goes low.
There are ...
Neil Turton

02/12/2016

10:46 PM Issue #1035 (Closed): Constructors and change detection does not honor --output-split-cfuncs
Two files completely dominate our C++ compilation times, and the two culprits seems to be two massive functions that ... Johan Bjork
08:13 PM Issue #1034: Extend V3Dead to be able to remove empty package modules/cells
Ignore the comments about Typedefs, that part of the change seems to be lost somewhere along the way. I'll likely fol... Johan Bjork
03:24 PM Issue #1034 (Closed): Extend V3Dead to be able to remove empty package modules/cells
Deletes Scopes, Typedefs, Cells pointing to empty modules and finally cells.
https://github.com/phb/verilator-dev/tr...
Johan Bjork

02/10/2016

07:30 PM Issue #1027: Partly out of range part-select gives wrong runtime result
I'll keep whacking away and see if I get somewhere.
More about adding various math operators to the Ast after V3Wi...
Johan Bjork
06:49 PM Issue #1027: Partly out of range part-select gives wrong runtime result
>Agreed, the generated code looks awful at the moment. I'm trying to think of specific optimizations that could be do... Wilson Snyder
06:42 PM Issue #1027: Partly out of range part-select gives wrong runtime result
Wilson Snyder wrote:
> 1) The spec says a select index is an integer. If we're going to do the right thing, do that...
Johan Bjork
06:29 PM Issue #1027: Partly out of range part-select gives wrong runtime result
Realized I broke one of the tests I have been caring about in the last upload, again relating to the question on how ... Johan Bjork
06:24 PM Issue #1027: Partly out of range part-select gives wrong runtime result
1) The spec says a select index is an integer. If we're going to do the right thing, do that.
Changing all of t...
Wilson Snyder
06:07 PM Issue #1027: Partly out of range part-select gives wrong runtime result
So I started with this, and it's a complete mess -- Any advice how to make this less terrible would be appreciated.
...
Johan Bjork
03:16 AM Issue #1033 (Resolved): Incorrect reversal of unpacked array
Two bugs related to same issue. Fixed in git towards 3.881.
A warning is hard, but there already is the LITENDIAN...
Wilson Snyder

02/09/2016

02:24 PM Issue #1033 (Closed): Incorrect reversal of unpacked array
Verilator fails to reverse unpacked arrays correctly. The test case is attached. A warning that the array is being ... Geoff Barrett
01:54 PM Issue #1030: Verilator compares this to null which is not valid c++
Sounds good to me. Thanks! Johan Bjork
03:32 AM Issue #1030 (Resolved): Verilator compares this to null which is not valid c++
I've committed all I'm going to fix for the time being, so will call this resolved for now as clang no longer will co... Wilson Snyder

02/06/2016

11:16 PM Issue #377: Support classes and methods
Does it make sense to enable classes given that interfaces are already supported and interfaces have similar function... Alex Solomatnikov

02/05/2016

01:48 AM Issue #1028 (Resolved): Core dump due to gate dedup graph recursion
The gate dedup logic was getting stuck, I added a recursion limit.
Fixed in git towards 3.881.
Wilson Snyder
12:25 AM Issue #1027: Partly out of range part-select gives wrong runtime result
I don't. Basically right now it looks at the msb/lsb range and makes a Bound check, which either turns on or turns o... Wilson Snyder
12:17 AM Issue #1027: Partly out of range part-select gives wrong runtime result
Did you have something in progress on this one (as mentioned in the linked ticket)? we have multiple designs impacted... Johan Bjork

02/04/2016

11:41 PM Issue #1032 (Resolved): elaboration time $display should not result in a warning
Thanks again! Pushed to git towards 3.881.
Wilson Snyder
02:26 PM Issue #1032 (Closed): elaboration time $display should not result in a warning
Follow up patch to bug973, when we switched from using EC_INFO to the new USERINFO class, there was no longer a way t... Johan Bjork
11:40 PM Issue #1031 (Resolved): read-after-free in V3Scope
Thanks again! Pushed to git towards 3.881.
Wilson Snyder
01:51 PM Issue #1031 (Closed): read-after-free in V3Scope
Ran valgrind on verilator to try and figure out a strange crash. Found this (unrelated to the crash) issue.
In V3S...
Johan Bjork
04:43 AM Issue #1021: avoid dynamic_cast for trivial casts
I didn't quite realize what amount of work that new warning would entail, thanks for looking into it. Didn't know it ... Johan Bjork
03:31 AM Issue #1021: avoid dynamic_cast for trivial casts
Note I may have to back this out due to bug1030 - it's illegal to test "this" for NULL, and it's important for code r... Wilson Snyder

02/03/2016

12:35 AM Issue #1030 (Assigned): Verilator compares this to null which is not valid c++
Working through these. Pushed fixing include's to trunk, so now verilated files are clean.
Wilson Snyder

02/02/2016

07:42 PM Issue #831: Can't replace lvalue assignments with const var
We just ran into the same issue.
As with the original reporter, I've had no luck reproducing this in a smaller desig...
Johan Bjork

02/01/2016

08:48 PM Issue #1030 (Closed): Verilator compares this to null which is not valid c++
... Johan Bjork

01/29/2016

04:12 AM Issue #1028: Core dump due to gate dedup graph recursion
A quick look shows the duplicate logic eliminator is blowing up the stack, something to do with all of the functions ... Wilson Snyder
03:50 AM Issue #1027 (Confirmed): Partly out of range part-select gives wrong runtime result
This is related to bug1008. The problem is fixing it requires significant runtime code which will may massively slow... Wilson Snyder
03:20 AM Issue #1029 (Closed): minor stats and debug print cleanup patch
Thanks, pushed to git towards 3.881. Wilson Snyder

01/27/2016

11:46 PM Issue #1029 (Closed): minor stats and debug print cleanup patch
https://github.com/phb/verilator-asserts/commits/master
Contains two commits:
1) Remove a UINFO(1,) that spams bu...
Johan Bjork
03:43 PM Issue #1028 (Closed): Core dump due to gate dedup graph recursion
Executing
verilator --trace -cc combined_main.v --exe sim_main.cpp
%Error: Verilator internal fault, sorry. C...
Jonathan Kimmitt
 

Also available in: Atom