We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
While rewriting some tests from chiseltest to ChiselSim, I've noticed ChiselEnum is not handled by the Peek/Poke API.
In a test with c.io.inst.peek() should be(inst) where inst is a ChiselEnum.Type, I get: value peek is not a member of chiselv.Instruction.Type.
c.io.inst.peek() should be(inst)
inst
ChiselEnum.Type
value peek is not a member of chiselv.Instruction.Type
The closest thing I could get to work is
c.io.inst.peekValue().asBigInt should be(inst.litValue)
Which is very convoluted and not user-friendly.
The text was updated successfully, but these errors were encountered:
Would this be the API you're looking for?
Sorry, something went wrong.
Having peek() on ChiselEnum would be nice... but it's not in merged into Chisel, right?
Not yet, but @kammoh has opened a draft PR adding it (among many other improvements): #4209
I am reviewing, but got sick over the weekend so am not quite at 100% 🙃
Oh, nice! just saw it mentioned! Commented here before I saw it.
Take care my friend! Hope you get better soon :)
No branches or pull requests
While rewriting some tests from chiseltest to ChiselSim, I've noticed ChiselEnum is not handled by the Peek/Poke API.
In a test with
c.io.inst.peek() should be(inst)
whereinst
is aChiselEnum.Type
, I get:value peek is not a member of chiselv.Instruction.Type
.The closest thing I could get to work is
Which is very convoluted and not user-friendly.
The text was updated successfully, but these errors were encountered: