Skip to content

Commit 1c72523

Browse files
committed
Xqci (Xqcilsm) extension: Fix IDL code and description of qc.setwm instruction to state that number of words written 0..31.
Signed-off-by: Albert Yosher <[email protected]>
1 parent ff7558a commit 1c72523

File tree

3 files changed

+16
-3
lines changed

3 files changed

+16
-3
lines changed

arch_overlay/qc_iu/ext/Xqci.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -288,6 +288,7 @@ versions:
288288
- Fix IDL code sign extension logic for qc.lieq instruction
289289
- Fix wrong mantissa bit selection in qc.norm, qc.normu and qc.normeu instructions
290290
- Fix wrong exponent calculation in qc.normeu instruction
291+
- Fix IDL code and description of qc.setwm instruction to state that number of words written 0..31.
291292
implies:
292293
- { name: Xqcia, version: "0.6.0" }
293294
- { name: Xqciac, version: "0.3.0" }
@@ -303,7 +304,7 @@ versions:
303304
- { name: Xqcili, version: "0.2.0" }
304305
- { name: Xqcilia, version: "0.2.0" }
305306
- { name: Xqcilo, version: "0.3.0" }
306-
- { name: Xqcilsm, version: "0.4.0" }
307+
- { name: Xqcilsm, version: "0.5.0" }
307308
- { name: Xqcisim, version: "0.2.0" }
308309
- { name: Xqcisls, version: "0.2.0" }
309310
- { name: Xqcisync, version: "0.2.0" }

arch_overlay/qc_iu/ext/Xqcilsm.yaml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,18 @@ versions:
5252
5353
changes:
5454
- Fix encoding of qc.swmi
55+
- version: "0.5.0"
56+
state: frozen
57+
ratification_date: null
58+
contributors:
59+
- name: Albert Yosher
60+
company: Qualcomm Technologies, Inc.
61+
62+
- name: Derek Hower
63+
company: Qualcomm Technologies, Inc.
64+
65+
changes:
66+
- Fix IDL code and description of qc.setwm instruction to state that number of words written 0..31.
5567
description: |
5668
The Xqcilsm extension includes six instructions that transfer multiple values
5769
between registers and memory.

arch_overlay/qc_iu/inst/Xqci/qc.setwm.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ name: qc.setwm
66
long_name: Set word multiple (Register)
77
description: |
88
Stores the value of `rs3` multiple times into the address starting at (`rs1` + `imm`).
9-
The number of writes is in `rs2`.
9+
The number of writes is in `rs2` bits [4:0] (0..31).
1010
Instruction encoded in R instruction format.
1111
definedBy:
1212
anyOf:
@@ -35,7 +35,7 @@ access:
3535
operation(): |
3636
XReg vaddr = X[rs1] + imm;
3737
Bits<32> write_value = X[rs3][31:0];
38-
XReg num_words = X[rs2];
38+
XReg num_words = X[rs2][4:0];
3939
for (U32 i = 0; i < num_words; i++) {
4040
write_memory<32>(vaddr, write_value, $encoding);
4141
vaddr = vaddr + 4;

0 commit comments

Comments
 (0)