-
Notifications
You must be signed in to change notification settings - Fork 22
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
Decoder attr #609
Decoder attr #609
Conversation
"vslide1up.vx", | ||
"vslidedown.vx", | ||
"vslideup.vx", | ||
"lsu", |
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.
lsu?
} | ||
|
||
case class isAdder(value: TriState) extends BooleanDecodeAttribute { | ||
override val description: String = "goes to [[org.chipsalliance.t1.rtl.LaneAdder]]." |
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.
Please provide a better documentation
|
||
def y(t1DecodePattern: T1DecodePattern): Boolean = { | ||
val allMatched = Seq( | ||
"vcompress.vm", |
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.
all of them are same, please dedup
case (fn, tri) if fn(t1DecodePattern) => isFloatmul(tri) | ||
}.get | ||
|
||
def y(t1DecodePattern: T1DecodePattern): Boolean = { |
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.
it should guarded by fp enabled, it can be accessed by T1DecodePattern.instruction
de6c45d
to
9acd11a
Compare
32c7a5c
to
a9ee870
Compare
72ff5e8
to
67110a2
Compare
884731f
to
bc191b1
Compare
c5bde54
to
fddef3c
Compare
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.
It needs to be more refactors in the future, but let's merge it.
No description provided.