dct_partial_test,fwd_txfm: change << to *

left shift of a negative number is undefined in C; quiets a ubsan
warning

Change-Id: Ib1624ad5326ac8e0eead9348468ef7fe5d4df9a4

diff --git a/test/dct_partial_test.cc b/test/dct_partial_test.cc
index 41f68955b23a7960de7dc55cc4f5f4bab5f4b03a..52c925cb783067bc55ecad89698283fb682fc4e2 100644 (file)
--- a/test/dct_partial_test.cc
+++ b/test/dct_partial_test.cc
@@ -46,7 +46,7 @@ tran_low_t partial_fdct_ref(const Buffer<int16_t> &in, int size) {
   }
 
   switch (size) {
-    case 4: sum <<= 1; break;
+    case 4: sum *= 2; break;
     case 8: /*sum = sum;*/ break;
     case 16: sum >>= 1; break;
     case 32: sum >>= 3; break;

diff --git a/vpx_dsp/fwd_txfm.c b/vpx_dsp/fwd_txfm.c
index aa59601094d7a5c8dd6aa519475442746b890087..6dcb3ba668143672fcbb1a4082d33910b9063525 100644 (file)
--- a/vpx_dsp/fwd_txfm.c
+++ b/vpx_dsp/fwd_txfm.c
@@ -84,7 +84,7 @@ void vpx_fdct4x4_1_c(const int16_t *input, tran_low_t *output, int stride) {
   for (r = 0; r < 4; ++r)
     for (c = 0; c < 4; ++c) sum += input[r * stride + c];
 
-  output[0] = sum << 1;
+  output[0] = sum * 2;
 }
 
 void vpx_fdct8x8_c(const int16_t *input, tran_low_t *final_output, int stride) {