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 staged generator PRs #2329

Merged
merged 3 commits into from
Mar 10, 2020
Merged

Revert staged generator PRs #2329

merged 3 commits into from
Mar 10, 2020

Conversation

debs-sifive
Copy link
Contributor

Related issue: #2322 #2309 #2274

Type of change:other enhancement

Impact: API modification - use prior Makefile conventions

Development Phase: implementation

Release Notes

This is causing backwards compatibility issues. Will be re-PRed in the future.

@debs-sifive debs-sifive requested a review from azidar March 10, 2020 20:54
@mwachs5
Copy link
Contributor

mwachs5 commented Mar 10, 2020

@debs-sifive
Copy link
Contributor Author

It should, but as Wes diagnosed, we will want to fix this in ChiselStage in the future, since we want to un-revert these changes ASAP as well.

Copy link
Contributor

@colinschmidt colinschmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a sane revert.

Looking forward to the new implementation.

@debs-sifive debs-sifive merged commit e170f8f into master Mar 10, 2020
@debs-sifive debs-sifive deleted the revert-stage branch March 11, 2020 20:34
debs-sifive added a commit that referenced this pull request Mar 16, 2020
debs-sifive added a commit that referenced this pull request Mar 25, 2020
debs-sifive added a commit that referenced this pull request Apr 6, 2020
* Restoring staged generator PRs (Reverting #2329)

This reverts commit e170f8f.

* bumping chisel 3.2.x
ucbjrl added a commit that referenced this pull request Apr 21, 2020
Revert staged generator PRs (#2329)

* Revert "updating README examples with Make changes (#2322)"

This reverts commit 875dcd3.

* Revert "fixing aspect generation (#2309)"

This reverts commit bbeb257.

* Revert "separate generator behavior into phases (#2274)"

This reverts commit 9d99750.

(cherry picked from commit e170f8f)
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.

3 participants