diff mbox series

[v3,03/31] target/arm/cpu.h: update comment for half-precision values

Message ID 20180223153636.29809-4-alex.bennee@linaro.org
State Superseded
Headers show
Series Add ARMv8.2 half-precision functions | expand

Commit Message

Alex Bennée Feb. 23, 2018, 3:36 p.m. UTC 
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

---
 target/arm/cpu.h | 1 +
 1 file changed, 1 insertion(+)

-- 
2.15.1

Comments

Richard Henderson Feb. 23, 2018, 11:30 p.m. UTC | #1 
On 02/23/2018 07:36 AM, Alex Bennée wrote:
> @@ -168,6 +168,7 @@ typedef struct {

>   *  Qn = regs[n].d[1]:regs[n].d[0]

>   *  Dn = regs[n].d[0]

>   *  Sn = regs[n].d[0] bits 31..0

> + *  Hn = regs[n].d[0] bits 15..0 for even n, and bits 31..16 for odd n


Everything past here --------------^ is wrong.


r~
diff mbox series

Patch

diff --git a/target/arm/cpu.h b/target/arm/cpu.h
index 267a9d7e2f..c2bce23fa5 100644
--- a/target/arm/cpu.h
+++ b/target/arm/cpu.h
@@ -168,6 +168,7 @@  typedef struct {
  *  Qn = regs[n].d[1]:regs[n].d[0]
  *  Dn = regs[n].d[0]
  *  Sn = regs[n].d[0] bits 31..0
+ *  Hn = regs[n].d[0] bits 15..0 for even n, and bits 31..16 for odd n
  *
  * This corresponds to the architecturally defined mapping between
  * the two execution states, and means we do not need to explicitly