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

Chisel type prints need improvement #953

Closed
ducky64 opened this issue Nov 29, 2018 · 0 comments
Closed

Chisel type prints need improvement #953

ducky64 opened this issue Nov 29, 2018 · 0 comments
Assignees
Milestone

Comments

@ducky64
Copy link
Contributor

ducky64 commented Nov 29, 2018

Printing Chisel types gives fun and useless strings like UInt@4fd5e7a. But the FIRRTL emitter can make coherent types, though that functionality isn't directly accessible from Data subtypes. What would be nice is for Data subtypes to override toString with:

  • as a chisel type: print the FIRRTL type, or something close to it, eg UInt<8>
  • as a bound object: also indicate the binding and parent module, ideally also add basic source locator information, eg UInt<8>(Reg in MyModule @ MyModule.scala:104)
  • as a literal: indicate the type and value, eg UInt<8>(233)

Bikeshedding on the precise representation is welcome.

Type of issue: feature request, other enhancement

Impact: no functional change

Development Phase: request, proposal

@jackkoenig jackkoenig added this to the 3.2.0 milestone Dec 17, 2018
@ducky64 ducky64 self-assigned this Jan 18, 2019
jackkoenig added a commit that referenced this issue Feb 28, 2023
Where the high form compiler removes Chirrtl (and runs some checks),
this compiler does nothing but read in the circuit and then emit it
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

No branches or pull requests

2 participants