Skip to content
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

Syntax error near "context". #847

Closed
mahdi259 opened this issue Mar 14, 2024 · 7 comments
Closed

Syntax error near "context". #847

mahdi259 opened this issue Mar 14, 2024 · 7 comments
Labels
HW hardware-related troubleshooting Something is not working as expected

Comments

@mahdi259
Copy link

Describe the bug
The updated testbench contains context keyword that is not supported in older software like ISE.

To Reproduce
I used neorv32_tb.vhd file as benchmark.

Expected behavior
Normal simulation.

Screenshots
image

@stnolting stnolting added HW hardware-related troubleshooting Something is not working as expected labels Mar 14, 2024
@mahdi259
Copy link
Author

Oh. I think this testbench file is only usable by VUnit, right?

@stnolting
Copy link
Owner

The testbench files in sim require VHDL-2008 (or newer) as standard. Please use the functional-equivalent "simple" testbench in sim/simple for toolchain that do not support this standard.

Btw, we really should add a note regarding this to the README 😅

@stnolting
Copy link
Owner

Oh. I think this testbench file is only usable by VUnit, right?

Well, yes, and VUnit requires VHDL-2008 (as far as I can see).

@mahdi259
Copy link
Author

Thanks for your reply.

@stnolting
Copy link
Owner

You're welcome! :)

@umarcor
Copy link
Collaborator

umarcor commented Mar 15, 2024

VUnit uses VHDL 2008 by default, but it does have a VHDL 1993 mode. The problem is that simulators from AMD/Xilinx traditionally do not support enough of VHDL 2008 to use the default, and neither support enough of VHDL 1993 to use the compatility mode. So, in order to use those tools, a non-standard variation of a 30y old version of the language needs to be used.

I would argue that it is less of a headache to use some simulator which does support VHDL 2008, such as GHDL or NVC. For synthesis, ghdl synth allows converting VHDL 2008 sources to some flavour that AMD/Xilinx can understand.

@stnolting
Copy link
Owner

For synthesis, ghdl synth allows converting VHDL 2008 sources to some flavour that AMD/Xilinx can understand.

Very good point!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HW hardware-related troubleshooting Something is not working as expected
Projects
None yet
Development

No branches or pull requests

3 participants