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

Bump master to chisel 3.6 #3354

Merged
merged 11 commits into from
Jun 15, 2023
Merged

Bump master to chisel 3.6 #3354

merged 11 commits into from
Jun 15, 2023

Conversation

jerryz123
Copy link
Contributor

No description provided.

@ZenithalHourlyRate
Copy link
Contributor

chisel 3.6 breaks mill. See chipsalliance/chisel#3045 (comment)

@jerryz123
Copy link
Contributor Author

chisel 3.6 breaks mill. See chipsalliance/chisel#3045 (comment)

What's the solution for this? The SBT release flow for 3.6.0 on master is fine for now.

@ZenithalHourlyRate
Copy link
Contributor

What's the solution for this? The SBT release flow for 3.6.0 on master is fine for now.

I assume backporting that fix to chisel 3.6.x branch and then releasing a 3.6.1 should work. @sequencer for this.

@sequencer
Copy link
Member

I think depending on maven release rather than compiling from source would help.

@ZenithalHourlyRate
Copy link
Contributor

chisel 3.6 breaks mill.

Oh I made a mistake here. I encountered such problem when working with a project that compiles chisel 3.6 from source. rocket-chip uses 3.6 from maven so things should be fine.

@jerryz123
Copy link
Contributor Author

@ZenithalHourlyRate or @sequencer can one of you update this PR with the fixes for mill?

@sequencer
Copy link
Member

I can do it after going back to keyboard tonight

@sequencer
Copy link
Member

I think mill doesn't break it since it already depends on the maven version of chisel

@jerryz123 jerryz123 changed the title Bump master to chisel 3.6 [WIP] Bump master to chisel 3.6 May 12, 2023
@jerryz123
Copy link
Contributor Author

Please don't merge this yet, we are encountering some issues with 3.6 bump

@ZenithalHourlyRate
Copy link
Contributor

CI is happy now. Failure on riscv-arch-test CI is irrelevant.

@ZenithalHourlyRate
Copy link
Contributor

we are encountering some issues with 3.6 bump

I assume we could merge now?

@jerryz123
Copy link
Contributor Author

Please wait, I found many bugs, I want to fix them for 3.5.6 first before bumping to 3.6.0

@jerryz123
Copy link
Contributor Author

We may also need to delay this until we have a suitable replacement for fixed-point in 3.6.0

@jackkoenig
Copy link
Contributor

We may also need to delay this until we have a suitable replacement for fixed-point in 3.6.0

FixedPoint is still supported in 3.6.0, it's just deprecated, you can suppress the deprecation warning. Just note that FixedPoint does not work in MFC so a suitable replacement that works in MFC would be very handy.

There are many interested in a Chisel-only FixedPoint implementation: chipsalliance/chisel#3161

@jerryz123
Copy link
Contributor Author

We may also need to delay this until we have a suitable replacement for fixed-point in 3.6.0

FixedPoint is still supported in 3.6.0, it's just deprecated, you can suppress the deprecation warning. Just note that FixedPoint does not work in MFC so a suitable replacement that works in MFC would be very handy.

There are many interested in a Chisel-only FixedPoint implementation: chipsalliance/chisel#3161

Oh great, that's good to hear, thanks. I'm losing track of what's getting dropped and what's getting deprecated each release.

@jerryz123 jerryz123 changed the title [WIP] Bump master to chisel 3.6 Bump master to chisel 3.6 Jun 15, 2023
@jerryz123 jerryz123 enabled auto-merge June 15, 2023 00:09
@jerryz123 jerryz123 merged commit d78198c into master Jun 15, 2023
@jerryz123 jerryz123 deleted the 3.6master branch June 15, 2023 00:10
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

Successfully merging this pull request may close these issues.

5 participants