-
Notifications
You must be signed in to change notification settings - Fork 241
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
Comments
Oh. I think this testbench file is only usable by VUnit, right? |
The testbench files in Btw, we really should add a note regarding this to the README 😅 |
Well, yes, and VUnit requires VHDL-2008 (as far as I can see). |
Thanks for your reply. |
You're welcome! :) |
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, |
Very good point! |
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](https://private-user-images.githubusercontent.com/73544027/312939129-d6b6cb32-3f5e-49f9-b2e5-55ec462ae7bc.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Mzk5MDE0NjgsIm5iZiI6MTczOTkwMTE2OCwicGF0aCI6Ii83MzU0NDAyNy8zMTI5MzkxMjktZDZiNmNiMzItM2Y1ZS00OWY5LWIyZTUtNTVlYzQ2MmFlN2JjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTglMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjE4VDE3NTI0OFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTI2Y2I2MGYyYzZlODRiNzc0OTIwMWI3ZjE2MmE5MGM3N2E5ZGJlZjU2ZTNkZGM2NzNmZmEwNmY1ZjVlNDE5ODYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.WNz4AAMS-wM88N-EkJAtQOPa83l4oBZgmbKlKWwqK1M)
The text was updated successfully, but these errors were encountered: