Xqci (Xqcia) extension: fix wrong mantissa bit selection for qc.norm, qc.normu and qc.normeu instructions

Signed-off-by: Albert Yosher <[email protected]>

Author: ayosher

arch_overlay/qc_iu/ext/Xqci.yaml

@@ -286,8 +286,9 @@ versions:
     - Fix IDL code and description increasing shift to 6 bit for qc.extdpr and qc.extdprh instructions
     - Fix IDL code and description increasing shift to 6 bit for qc.extdupr and qc.extduprh instructions
     - Fix IDL code sign extension logic for qc.lieq instruction
+    - Fix wrong mantissa bit selection in qc.norm, qc.normu and qc.normeu instructions
   implies:
-  - { name: Xqcia, version: "0.5.0" }
+  - { name: Xqcia, version: "0.6.0" }
   - { name: Xqciac, version: "0.3.0" }
   - { name: Xqcibi, version: "0.2.0" }
   - { name: Xqcibm, version: "0.7.0" }

arch_overlay/qc_iu/ext/Xqcia.yaml

@@ -67,6 +67,18 @@ versions:
     - Fix typos in description of qc.shlsat and qc.shlusat instructions
     - Fix bug in qc.shlsat that caused wrong IDL code result
     - Fix clobbering of saturation results in [add|addu|sub]sat instructions
+- version: "0.6.0"
+  state: frozen
+  ratification_date: null
+  contributors:
+  - name: Albert Yosher
+    company: Qualcomm Technologies, Inc.
+    email: [email protected]
+  - name: Derek Hower
+    company: Qualcomm Technologies, Inc.
+    email: [email protected]
+  changes:
+    - Fix wrong mantissa bit selection in qc.norm, qc.normu and qc.normeu instructions
 description: |
   The Xqcia extension includes eleven instructions to perform integer arithmetic.
 

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

@@ -35,4 +35,4 @@ operation(): |
   XReg clo = (xlen() - 1) - $signed(highest_set_bit(~X[rs1]));
   XReg mnt = (X[rs1][31] == 1) ? (X[rs1] << (clo - 1)) : (X[rs1] << (clz - 1));
   XReg exp = (X[rs1][31] == 1) ? (-(clo - 1)) : (-(clz - 1));
-  X[rd] = {mnt[23:0],exp[7:0]};
+  X[rd] = {mnt[31:8],exp[7:0]};

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

@@ -34,4 +34,4 @@ operation(): |
   XReg clz = ((xlen() - 1) - $signed(highest_set_bit(X[rs1]))) & ~1;
   XReg mnt = (X[rs1] << (clz & ~1));
   XReg exp = ((-clz) & ~1);
-  X[rd] = {mnt[23:0],exp[7:0]};
+  X[rd] = {mnt[31:8],exp[7:0]};

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

@@ -34,4 +34,4 @@ operation(): |
   XReg clz = (xlen() - 1) - $signed(highest_set_bit(X[rs1]));
   XReg mnt = (X[rs1] << clz);
   XReg exp = (-clz);
-  X[rd] = {mnt[23:0],exp[7:0]};
+  X[rd] = {mnt[31:8],exp[7:0]};