EnricoGiordano1992 / resim-simulating-partial-reconfiguration

Automatically exported from code.google.com/p/resim-simulating-partial-reconfiguration
Other
0 stars 0 forks source link

ReSim-1: Incorrect variable type used in get_config_interface and set_config_interface #1

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago

Users report the following error message when running the "XDRS.QUICKSTART" 
example. After executing the "simulate_mti.do" script, ModelSim reports:

Error: ./artifacts/my_region.sv(277): (vlog-2244) Variable 'wrp' is implicitly 
static. You must either explicitly declare it as static or automatic
# 
# or remove the initialization in the declaration of variable.
# 
# ** Error: C:/Modeltech_pe_edu_10.2c/win32pe_edu/vlog failed.
# Error in macro ./simulate_mti.do line 30

ReSim: 2.3b
ModelSim: 10.2c
OS: Win32

Original issue reported on code.google.com by george.g...@gmail.com on 31 May 2014 at 4:50

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
This error message does not exist on ModelSim 6.5g but is visible on later 
versions of ModelSim. Please modify the following lines and try again.

...ReSim/src/rsv_defines.svh (57-58): 

- ovm_object obj; \ 
- wrapper_ wrp; \
+ automatic ovm_object obj; \
+ automatic wrapper_ wrp; \

...ReSim/src/rsv_defines.svh (65):

 - wrapper_ wrp = new (tag_, vi_); \
 + automatic wrapper_ wrp = new (tag_, vi_); \

The changes add "automatic" key words to the "wrp" variables. Please make sure 
that you have not deleted the "\" at the end of these lines. 
Verilog/SystemVerilog requires to add "\" for multiple macros.

We will aim to fix this issue at later versions of ReSim release.

Original comment by george.g...@gmail.com on 1 Jun 2014 at 12:19

GoogleCodeExporter commented 8 years ago
The above fix has been committed to the project tree, although it has not yet 
been officially released. We aim to release the above fix in ReSim 2.3c . 

You can checkout the latest development of ReSim from here 
(https://code.google.com/p/resim-simulating-partial-reconfiguration/source/check
out). The ReSim project uses git as the version control tool. Please refer to 
this page (http://git-scm.com/) to get "git". The git commit hash that has this 
fix is "16bb0be97255"

Original comment by george.g...@gmail.com on 26 Jul 2014 at 8:45