ref: 556780b7556b5be83ab49d75b23f74434f848132
parent: 5d888dde5422d34ff0847ad14352943564f49902
author: Martin Storsjö <martin@martin.st>
date: Sun Apr 7 20:13:20 EDT 2019
arm: Fix typos in comments The width register has been set to clz(w)-24, not the other way around. And the 32 bit prep function has got the h parameter in r4, not in r5.
--- a/src/arm/32/mc.S
+++ b/src/arm/32/mc.S
@@ -217,7 +217,7 @@
// This has got the same signature as the put_8tap functions,
// assumes that the caller has loaded the h argument into r5,
-// and assumes that r8 is set to (24-clz(w)).
+// and assumes that r8 is set to (clz(w)-24).
function put
adr r9, L(put_tbl)
ldr r8, [r9, r8, lsl #2]
@@ -307,8 +307,8 @@
// This has got the same signature as the put_8tap functions,
-// assumes that the caller has loaded the h argument into r5,
-// and assumes that r8 is set to (24-clz(w)), and r7 to w*2.
+// assumes that the caller has loaded the h argument into r4,
+// and assumes that r8 is set to (clz(w)-24), and r7 to w*2.
function prep
adr r9, L(prep_tbl)
ldr r8, [r9, r8, lsl #2]
--- a/src/arm/64/mc.S
+++ b/src/arm/64/mc.S
@@ -235,7 +235,7 @@
// This has got the same signature as the put_8tap functions,
-// and assumes that x8 is set to (24-clz(w)).
+// and assumes that x8 is set to (clz(w)-24).
function put
adr x9, L(put_tbl)
ldrh w8, [x9, x8, lsl #1]
@@ -330,7 +330,7 @@
// This has got the same signature as the prep_8tap functions,
-// and assumes that x8 is set to (24-clz(w)), and x7 to w*2.
+// and assumes that x8 is set to (clz(w)-24), and x7 to w*2.
function prep
adr x9, L(prep_tbl)
ldrh w8, [x9, x8, lsl #1]