tree 0e0866f7b932040c37937e59069983630dd86f1c
parent 8a40f783702f9946ee767f55bbefe369e483769e
author James Zern <jzern@google.com> 1679966918 -0700
committer James Zern <jzern@google.com> 1680114706 +0000

dr_prediction_z2_Nx8_sse4_1: quiet -Wmaybe-unintialized warnings

Reorder some operations to avoid unneeded work when there have been no y
calculations (base_x >= min_base_x).

This change is similar to the one done for NEON:
0484e4d9f5 dr_prediction_z2_Nx8_neon: quiet -Wunintialized warnings

Quiets the following under gcc 12.2.0 targeting x86:
In function ‘_mm_mullo_epi16’,
    inlined from ‘dr_prediction_z2_Nx8_sse4_1’ at aom_dsp/x86/intrapred_sse4.c:700:9,
    inlined from ‘av1_dr_prediction_z2_sse4_1’ at aom_dsp/x86/intrapred_sse4.c:921:7:
/usr/lib/gcc/x86_64-linux-gnu/12/include/emmintrin.h:1162:35: warning:
‘shifty’ may be used uninitialized [-Wmaybe-uninitialized]
 1162 |   return (__m128i) ((__v8hu)__A * (__v8hu)__B);
      |                                   ^~~~~~~~~~~
aom_dsp/x86/intrapred_sse4.c: In function ‘av1_dr_prediction_z2_sse4_1’:
aom_dsp/x86/intrapred_sse4.c:605:34: note: ‘shifty’ was declared here
  605 |     __m128i b, res, res1, shift, shifty;
      |

Change-Id: I4d414fea6cdd4d76c6dff0ce6f1fb25ba9c04a54