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

OSVVM update #1046

Open
c-thaler opened this issue Aug 2, 2024 · 9 comments
Open

OSVVM update #1046

c-thaler opened this issue Aug 2, 2024 · 9 comments

Comments

@c-thaler
Copy link

c-thaler commented Aug 2, 2024

OSVVM is currently rev-tied to a revision that is over a year old. In the meantime, there was a nice feature added that comes in handy when implementing bus masters (DelayCoverageIDType in DelayCoveragePkg.vhd).
It would be nice to have the rev-tie get bumped to the latest stable commit.

LarsAsplund added a commit that referenced this issue Aug 8, 2024
InitSeed changed behavior such that randomized testbenches don't reproduce the previously
used stimuli. One testbench had to be updated to compensate.
@LarsAsplund
Copy link
Collaborator

@c-thaler I've pushed an update but I have yet to test over all simulators before merging.

@c-thaler
Copy link
Author

c-thaler commented Aug 8, 2024

I will go one with the branch until it is merged. Thanks a lot!

@LarsAsplund
Copy link
Collaborator

Looks like NVC fails to compile this OSVVM version. @nickg do you know anything about this?

@nickg
Copy link
Contributor

nickg commented Aug 9, 2024

Looks like NVC fails to compile this OSVVM version. @nickg do you know anything about this?

This was reported and fixed in nickg/nvc#917. Can you update the CI to use the 1.13.1 release?

@LarsAsplund
Copy link
Collaborator

Will do. Thanks!

@LarsAsplund
Copy link
Collaborator

@nickg The latest NVC fixed the problem but there is a new ambiguity problem. I'm on my cell phone so I haven't been able to test any workaround. Is that really ambitious?

@nickg
Copy link
Contributor

nickg commented Aug 9, 2024

No it's a bug, I'll fix it this weekend. Sorry about that.

@nickg
Copy link
Contributor

nickg commented Aug 11, 2024

No it's a bug, I'll fix it this weekend. Sorry about that.

This is fixed in the latest 1.13.2 release.

@LarsAsplund
Copy link
Collaborator

Thanks @nickg, I will rerun the test once the new NVC image is uploaded.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants