-
Notifications
You must be signed in to change notification settings - Fork 271
Should lmul[2:0] be contiguous in vtype? #440
Comments
Is that "backwards compatibility with pre-freeze drafts" going to remain with us forever like in some other specs? |
This was a local revert to reduce churn in implementations. This issue is exactly about whether to make contiguous in final spec. There will undoubtedly be some other incompatible changes in spec before 1.0 is frozen, this is about reducing the number of intermediate updates implementation teams have to do. |
I would vote yes on aesthetic grounds. My question is whether you intend V-ext to be extensible to allow additional LMUL or SEW values, and, if so, where these bits would go. |
The immediate in |
I vote no, unless there is some other compelling reason to make another backwards incompatible change to this register. I agree on the aesthetics, but it’s not enough for me. |
Apart from asthetics, software emulators have to concatenate those scattered fields. |
I think we already crossed the bridge of not optimizing for software emulators when we chose the branch/jump and RVC immediate encodings... |
Group decided to regroup bits for v1.0 release. |
UPDATE: I reversed my stance (because I erred).
ORIGINAL POST:
|
I have to say 👎 to this. It’s more disruptive now than it was during the 0.9 transition, so the justification for keeping it constant for 0.9 applies more to 1.0, not less.
|
I agree with @aswaterman that reserving the bit between sew and vlmul is inappropriate. |
For backwards compatibility with earlier tool chains and implementations, reverted change to make expanded lmul contiguous (#432).
Should we reformat to make lmul contiguous before base 1.0?
The text was updated successfully, but these errors were encountered: