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

Revert "Yosys Update -> v0.42" #2643

Merged
merged 1 commit into from
Jul 10, 2024
Merged

Revert "Yosys Update -> v0.42" #2643

merged 1 commit into from
Jul 10, 2024

Conversation

AlexandreSinger
Copy link
Contributor

Reverts #2596

The PR was merged at a point when it only turned off testcases without pushing code which broke them.

The testcases should be re-enabled until the code that needs them off is merged.

@github-actions github-actions bot added the infra Project Infrastructure label Jul 9, 2024
@vaughnbetz
Copy link
Contributor

Thanks Alex. Adding @amirarjmand93 so he's aware of this one. My bad for merging a PR that I shouldn't have!

@AlexandreSinger
Copy link
Contributor Author

@vaughnbetz This has passed CI. Please merge when you are available.

@vaughnbetz
Copy link
Contributor

Thanks Alex!

@vaughnbetz vaughnbetz merged commit 16adbfa into master Jul 10, 2024
53 of 101 checks passed
@vaughnbetz vaughnbetz deleted the revert-2596-yosys_update branch July 10, 2024 01:35
@amirarjmand93
Copy link
Contributor

@vaughnbetz I am working to turning f4pga tests off here. i am trying to disable System Verilog tasks. I don't want to disable the entire test.( more details will be reported)
but getting some failures on Parmys is annoying. in Parmys basic test and Parmys_reg_strong[DYOSYS_F4PGA_PLUGINS=OFF]. my major concern is Parmys tests that should be tackled and updated.

in addition, two new qor failures on other tests have been issued. /vtr_reg_basic/basic_timing and vtr_reg_strong/strong_power. I extracted their xlsx sheets and I put them on PR tomorrow.

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

Successfully merging this pull request may close these issues.

3 participants