-
Notifications
You must be signed in to change notification settings - Fork 125
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
VLEN should not be set by preprocessor macro in ara_pkg.sv #293
Comments
Hello @jerryz123, you are right. We have done it in the beginning to speed up the development but now it's really ugly. |
Solving in #317 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This makes integration in other projects cumbersome. VLEN should follow same pattern as NR_LANES... ara_tb should set the VLEN parameter from the macro, but ara_pkg should not reference the macro.
The text was updated successfully, but these errors were encountered: