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.

Sign up for GitHub

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

Jump to bottom

Fix VCS incompatible type issue #2589

Open

murattokez wants to merge 1 commit into openhwgroup:cv32e40p/dev from murattokez:dev

murattokez commented Mar 7, 2025

Summary

This PR fixes an issue where VCS throws an "Incompatible complex type" error. The fix ensures compatibility with both Questa and VCS.

Changes

Updated cv32e40p_load_store_instr_lib.sv to assign the dynamic array to a temporary queue before usage.
Applied the same fix to cv32e40p_loop_instr.sv.
Verified that the fix works on both Questa and VCS (tested with V-2023.12-SP1-1).

Related Issues

Similar issue observed in Incorporate UNSUPPORTED_WITH kludge for VCS #2393
Related PR: Remove UNSUPPORTED_WITH workaround for VCS (resolved as of V-2023.12-1) #2402 (where an earlier workaround was removed)

Testing

✅ Verified compilation with Questa.
✅ Verified compilation with VCS (V-2023.12-SP1-1).
No functional changes expected.

murattokez closed this

murattokez reopened this

murattokez closed this

murattokez reopened this


          Fix VCS incompatible type issue

ceb5544

murattokez force-pushed the dev branch from cb589db to ceb5544 Compare

March 7, 2025 05:45

murattokez closed this

murattokez reopened this

murattokez closed this

murattokez reopened this

murattokez closed this

murattokez reopened this

murattokez closed this

murattokez reopened this

MikeOpenHWGroup self-requested a review

March 7, 2025 14:44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet