-
Notifications
You must be signed in to change notification settings - Fork 589
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
Add a Select.ios that works with Definition/Instance, fix isA behavior #2315
Conversation
core/src/main/scala/chisel3/experimental/hierarchy/Instance.scala
Outdated
Show resolved
Hide resolved
2744479
to
43e29dd
Compare
Although this is a feature addition I would like to backport to 3.5.x |
I need to add a test |
println("```") | ||
val y = chisel3.stage.ChiselStage.emitFirrtl(new InOutTop) | ||
println("```") | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent example!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, I do have a couple of requests though.
src/test/scala/chiselTests/experimental/hierarchy/InstanceSpec.scala
Outdated
Show resolved
Hide resolved
src/test/scala/chiselTests/experimental/hierarchy/InstanceSpec.scala
Outdated
Show resolved
Hide resolved
src/test/scala/chiselTests/experimental/hierarchy/InstanceSpec.scala
Outdated
Show resolved
Hide resolved
Co-authored-by: Jack Koenig <koenig@sifive.com>
….scala Co-authored-by: Jack Koenig <koenig@sifive.com>
….scala Co-authored-by: Jack Koenig <koenig@sifive.com>
…tance-select-tweaks
Contributor Checklist
docs/src
?Type of Improvement
API Impact
Adds a Select.ios function that works with Hierarchy (
Definition
orInstance
). This effectively exposes the.proto
's.ios
as part of the public API for Select.Backend Code Generation Impact
No backend generation change as this was not available before.
Desired Merge Strategy
Release Notes
Add a Select.ios function that works with Hierarchy (
Definition
orInstance
). Fix a bug in.isA
for Select APIs for Definition/Instance.Reviewer Checklist (only modified by reviewer)
3.3.x
, [small] API extension:3.4.x
, API modification or big change:3.5.0
)?Please Merge
?