Open veripoolbot opened 5 years ago
Original Redmine Comment Author Name: Wilson Snyder (@wsnyder) Original Date: 2019-06-04T00:53:39Z
Verilog-mode doesn't figure out how to deal with 2-D and up structures, it just assumes the first output is what you want. The combining code is already complicated so is unlikely to get reworked to improve this case, so you'll need to declare it manually.
Original Redmine Comment Author Name: David Rogoff Original Date: 2019-10-11T17:14:22Z
Wilson, I understand you not wanting to fix this but I keep hitting this issue on huge upper-level integration modules - exactly where AUTOs are great. Have you considered making a paid version of verilog-mode to give you and other developers incentive to do some of the more difficult fixes? After years and years of relying on your code (and emacs) I'm starting to look at eclipse with plugins like DVT (which is certainly not free!).
David
Original Redmine Comment Author Name: Wilson Snyder (@wsnyder) Original Date: 2019-10-18T02:24:29Z
If you can provide some examples I'll take a look if there's some straightforward cases I can cover (gratis). Please include a reasonable number of cases in a file along with how you think they should be properly combined, including some cases that shouldn't combine. Note presently all the following are different internally so each of these need to be tested against whatever combine permutations:
reg [x:y] packed;
reg [x:y][x:y] packed2d;
reg unpacked [x:y];
reg unpacked2d [x:y][x:y];
reg [x:y] packedunpacked [x:y];
reg [x:y][x:y] unpacked2d [x:y][x:y];
@veripoolbot , @wsnyder is it fixed? Basically I am having problem while using 2D packed array. reg [x:y][x:y] packed2d; is always coming as reg [0][x:y] packed2d; for the output
If a bug is open it is not fixed.
Author Name: David Rogoff Original Redmine Issue: 1453 from https://www.veripool.org
Hi Wilson.
Thanks for the super-quick answer to my last question!
Here's another:
I have a signal like this: logic [7:0] sig_a [2];
comes from one instance of module xx and sig_a[1] from the other instance.
The full ```sig_a [2]