Update integrated-matrix.adoc#35
Merged
joseemoreira merged 19 commits intoriscv:integrated-matrix-extensionfrom Apr 22, 2026
Merged
Update integrated-matrix.adoc#35joseemoreira merged 19 commits intoriscv:integrated-matrix-extensionfrom
joseemoreira merged 19 commits intoriscv:integrated-matrix-extensionfrom
Conversation
Update integrated-matrix.adoc Revised the arithmetic considerations section for floating-point. Semantics is defined based on two implementation-defined parameters: G and psm. Those parameters must be defined (off-band) by the architecture as a table of (lambda, SEW, W) -> (G, psm) mappings. This is a compromise between implementation freedom and complete architectural specification. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
efocht-oct
reviewed
Apr 17, 2026
| stem:[C_{i,j} \leftarrow C_{i,j} + \sum_{k=0}^{\lambda W - 1} A_{i,k} \times B_{k,j}] | ||
|
|
||
| An implementation partitions the λ sub-dot-products for each output element into consecutive groups of `G` sub-dot-products. | ||
| The implementation-defined grouping factor `G` is applied separately within each `LMUL=1` step, so that a group shall not cross the boundary between two consecutive `LMUL=1` steps. |
There was a problem hiding this comment.
We didn't have this limitation before.
Collaborator
Author
There was a problem hiding this comment.
What limitation you refer to?
ptomsich
requested changes
Apr 17, 2026
Collaborator
ptomsich
left a comment
ptomsich
left a comment
There was a problem hiding this comment.
Request dropping PSM as a normative, prescriptive specification. It is declaratory only.
| | `vmmacc.vv vd, vs1, vs2` | 1 | SEW | SEW | ||
| | `vwmmacc.vv vd, vs1, vs2` | 2 | SEW/2 | SEW | ||
| | `vqwmmacc.vv vd, vs1, vs2` | 4 | SEW/4 | SEW | ||
| | `vqmmacc.vv vd, vs1, vs2` | 4 | SEW/4 | SEW |
Collaborator
There was a problem hiding this comment.
As discussed: a pain, but ok.
We need to still mention somewhere that vqwmmacc is a valid alias, but vqmmacc is the canonical disassembly.
Collaborator
Author
There was a problem hiding this comment.
Hi Philipp, could you please elaborate a bit? Why do we need the qw alias? I thought you found a precendet for using just q for quad-widening.
Tightening language of arithmetic considerations a bit. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
Tightened language a bit. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
Updated description of `psm`. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
Tightened the description of dot-product across G groups. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
Tightened language a bit. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
Editing characters. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
typos Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
Editing characters. Signed-off-by: Jose Moreira <jmoreira@us.ibm.com>
joseemoreira
merged commit 226e30e
into
riscv:integrated-matrix-extension
2 of 3 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update integrated-matrix.adoc
Revised the arithmetic considerations section for floating-point. Semantics is defined based on two implementation-defined parameters: G and psm. Those parameters must be defined (off-band) by the architecture as a table of (lambda, SEW, W) -> (G, psm) mappings.
This is a compromise between implementation freedom and complete architectural specification.
Please also check the "Specialized Extensions" section.