[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
  SystemPerl
  Voneline
  WFH
General Info
  Papers

Issue #448

Support array querying system functions

Added by Iztok Jeras about 1 year ago. Updated 4 months ago.

Status:Closed Start date:03/06/2012
Priority:Low Due date:
Assignee:Wilson Snyder % Done:

0%
Category:Unsupported
Target version:-

Description

Hi,

Support for the the next SystemVerilog array querying methods is missing: $dimensions $left $right $low $high $increment $size

The next example can be used to test the feature: test_regress/t/t_array_packed_methods.v the test can be found at (branch test_sv): https://github.com/jeras/verilator

Regards, Iztok Jeras

History

#1
Updated by Wilson Snyder about 1 year ago

  • Category set to Unsupported
  • Status changed from New to Assigned
  • Assignee set to Wilson Snyder

Do these synthesize?

#2
Updated by Iztok Jeras about 1 year ago

Hi Wilson,

I did not check the standard, but I think they do synthesize. They are constant functions, at least for arrays with fixed sizes. They should behave similarly to $clog2 or $bits. But you are correct in questioning their usefulness in synthesizable code, I do not see many usage scenarios, in most cases I would use the same parameters used in the array declaration instead of this functions. They could be placed on a list of unsupported SystemVerilog keywords, and the bug can be closed?

Regards, Iztok Jeras

#3
Updated by Wilson Snyder about 1 year ago

  • Subject changed from array querying methods to Support array querying methods
  • Priority changed from Normal to Low

Given your notes I'll just make it lower priority. Shouldn't take long to add once structs are in.

#4
Updated by Iztok Jeras about 1 year ago

Hi,

This issue should be renamed to "Support array querying system functions".

If updated tests are merged from GitHub the new test name would be: test_regress/t/t_array_packed_sysfunct.v

Regards, Iztok Jeras

#5
Updated by Wilson Snyder about 1 year ago

  • Subject changed from Support array querying methods to Support array querying system functions

#6
Updated by Wilson Snyder 4 months ago

  • Status changed from Feature to Closed

Fixed in git towards 3.845.

Also available in: Atom