commit 8a045659758ca7f02a1f8daa94b3fbe82802b725
author Marat Dukhan <maratek@google.com> Tue Jul 20 14:07:21 2021 -0700
committer XNNPACK Team <xnnpack-github-robot@google.com> Tue Jul 20 14:08:09 2021 -0700
tree 418f0fc9aade8d6b3378a3d1a8c61d42ff916771
parent c3f69fd3d543cdffbbf9e34630d3da66b22589fc

Use RNDNU requantization in QS8 VADD[C] microkernels

Rounding midpoints up (RNDNU) is more efficient than previously used RNDNA
(rounding midpoints away from zero) requantization

PiperOrigin-RevId: 385867165

src/qs8-vaddc/gen/minmax-avx-mul16-ld64-x16.c

diff --git a/src/qs8-vaddc/gen/minmax-avx-mul16-ld64-x16.c b/src/qs8-vaddc/gen/minmax-avx-mul16-ld64-x16.c
index c8bd2bf..4d619dd 100644
--- a/src/qs8-vaddc/gen/minmax-avx-mul16-ld64-x16.c
+++ b/src/qs8-vaddc/gen/minmax-avx-mul16-ld64-x16.c
@@ -54,19 +54,10 @@
     __m128i vacc89AB = _mm_add_epi32(vbias, _mm_unpacklo_epi16(vaprod89ABCDEFlo, vaprod89ABCDEFhi));
     __m128i vaccCDEF = _mm_add_epi32(vbias, _mm_unpackhi_epi16(vaprod89ABCDEFlo, vaprod89ABCDEFhi));
 
-    const __m128i vadj0123 = _mm_srai_epi32(vacc0123, 31);
-    vacc0123 = _mm_add_epi32(vacc0123, vrounding);
-    const __m128i vadj4567 = _mm_srai_epi32(vacc4567, 31);
-    vacc4567 = _mm_add_epi32(vacc4567, vrounding);
-    const __m128i vadj89AB = _mm_srai_epi32(vacc89AB, 31);
-    vacc89AB = _mm_add_epi32(vacc89AB, vrounding);
-    const __m128i vadjCDEF = _mm_srai_epi32(vaccCDEF, 31);
-    vaccCDEF = _mm_add_epi32(vaccCDEF, vrounding);
-
-    vacc0123 = _mm_sra_epi32(_mm_add_epi32(vacc0123, vadj0123), vshift);
-    vacc4567 = _mm_sra_epi32(_mm_add_epi32(vacc4567, vadj4567), vshift);
-    vacc89AB = _mm_sra_epi32(_mm_add_epi32(vacc89AB, vadj89AB), vshift);
-    vaccCDEF = _mm_sra_epi32(_mm_add_epi32(vaccCDEF, vadjCDEF), vshift);
+    vacc0123 = _mm_sra_epi32(_mm_add_epi32(vacc0123, vrounding), vshift);
+    vacc4567 = _mm_sra_epi32(_mm_add_epi32(vacc4567, vrounding), vshift);
+    vacc89AB = _mm_sra_epi32(_mm_add_epi32(vacc89AB, vrounding), vshift);
+    vaccCDEF = _mm_sra_epi32(_mm_add_epi32(vaccCDEF, vrounding), vshift);
 
     __m128i vout01234567 = _mm_adds_epi16(_mm_packs_epi32(vacc0123, vacc4567), voutput_zero_point);
     __m128i vout89ABCDEF = _mm_adds_epi16(_mm_packs_epi32(vacc89AB, vaccCDEF), voutput_zero_point);
@@ -98,13 +89,8 @@
       __m128i vacc0123 = _mm_add_epi32(vbias, _mm_unpacklo_epi16(vaprod01234567lo, vaprod01234567hi));
       __m128i vacc4567 = _mm_add_epi32(vbias, _mm_unpackhi_epi16(vaprod01234567lo, vaprod01234567hi));
 
-      const __m128i vadj0123 = _mm_srai_epi32(vacc0123, 31);
-      const __m128i vadj4567 = _mm_srai_epi32(vacc4567, 31);
-      vacc0123 = _mm_add_epi32(vacc0123, vrounding);
-      vacc4567 = _mm_add_epi32(vacc4567, vrounding);
-
-      vacc0123 = _mm_sra_epi32(_mm_add_epi32(vacc0123, vadj0123), vshift);
-      vacc4567 = _mm_sra_epi32(_mm_add_epi32(vacc4567, vadj4567), vshift);
+      vacc0123 = _mm_sra_epi32(_mm_add_epi32(vacc0123, vrounding), vshift);
+      vacc4567 = _mm_sra_epi32(_mm_add_epi32(vacc4567, vrounding), vshift);
 
       __m128i vout01234567 = _mm_adds_epi16(_mm_packs_epi32(vacc0123, vacc4567), voutput_zero_point);
       vout01234567 = _mm_max_epi16(vout01234567, voutput_min);