-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How to run unisim using cxxrtl #197
Comments
Apparently, there are some global signals which aren't yet supported. You could probably try to modify unisim to remove them. If you have a reproducer (not using unisim, which is not open source), I could try to add support for it. |
Can you suggest some open source reproducer. |
|
That's what's being asked of you. For the issue to be fixed, there needs to be a design that reproduces the issue without using proprietary libraries - you're in the best position to create such a design, since you just need to modify your existing reproducer to not use unisim. |
If I remember correctly, the GSR/GTS signals are only necessary for simulation and where generated like shown here. Did you include |
Is signal in package is allowed in vhdl. |
Yes, signal declarations are allowed in package according to the VHDL LRM. |
yosys -m ghdl -p "ghdl --std=93c --ieee=synopsys -fexplicit -Whide -Wspecs dut2; write_cxxrtl blink.cpp"
/----------------------------------------------------------------------------
| yosys -- Yosys Open SYnthesis Suite |
| Copyright (C) 2012 - 2024 Claire Xenia Wolf [email protected] |
| Distributed under an ISC-like license, type "license" to see terms |
----------------------------------------------------------------------------/
Yosys 0.42+28 (git sha1 ede3750a6, g++ 10.2.0 -fPIC -Os)
-- Running command `ghdl --std=93c --ieee=synopsys -fexplicit -Whide -Wspecs dut2; write_cxxrtl blink.cpp' --
/tools/Xilinx/Vivado/2019.2/data/vhdl/src/unisims/primitive/FIFO18E2.vhd:420:27:error: no declaration for "gsr"
glblGSR <= TO_X01(GSR);
^
ERROR: vhdl import failed.
The text was updated successfully, but these errors were encountered: