commit 34e1201ad5a5154fde772ddbe37d32d985c559b8
author Nathan E. Egge <negge@mozilla.com> Wed Sep 13 09:02:32 2017 -0400
committer Nathan Egge <negge@mozilla.com> Fri Sep 15 23:58:12 2017 +0000
tree 7d5ce9950b8f27f48dc0a5a8e5a51b6552b249bf
parent 22fe4a139ac244983bf4b8dbc740b5d22f518487

Force C implementation of 16-point Daala TX's.

This patch fixes a regression introduced in 1d190950 where the encoder
 was using the 16x16 VP9/AV1 transforms for RDO, but then used the Daala
 transforms for encoding.

subset1:

master-daala_dct16@2017-09-13T12:05:18.013Z ->
  master_daala_dct16_use_c@2017-09-13T13:05:02.252Z

   PSNR | PSNR Cb | PSNR Cr | PSNR HVS |    SSIM | MS SSIM | CIEDE 2000
-0.3654 | -0.7634 | -0.7407 |  -0.4884 | -0.4699 | -0.4945 |    -0.5104

master-no_rect_tx-no_var_tx@2017-09-12T00:23:18.153Z ->
  master_daala_dct16_use_c@2017-09-13T13:05:02.252Z

   PSNR | PSNR Cb | PSNR Cr | PSNR HVS |    SSIM | MS SSIM | CIEDE 2000
-0.0133 |  0.1040 | -0.0440 |  -0.0492 | -0.0151 | -0.0120 |     0.0699

Change-Id: Id1830d0975db4bd0320a47fdf45b4bca20881cfb

av1/common/idct.c

diff --git a/av1/common/idct.c b/av1/common/idct.c
index 7e3bdb2..086b6e2 100644
--- a/av1/common/idct.c
+++ b/av1/common/idct.c
@@ -1843,7 +1843,11 @@
     case ADST_DCT:
     case DCT_ADST:
     case ADST_ADST:
+#if CONFIG_DAALA_DCT16
+      av1_iht16x16_256_add_c(input, dest, stride, txfm_param);
+#else
       av1_iht16x16_256_add(input, dest, stride, txfm_param);
+#endif  // CONFIG_DAALA_DCT16
       break;
 #if CONFIG_EXT_TX
     case FLIPADST_DCT:
@@ -1857,7 +1861,11 @@
     case H_ADST:
     case V_FLIPADST:
     case H_FLIPADST:
+#if CONFIG_DAALA_DCT16
+      av1_iht16x16_256_add_c(input, dest, stride, txfm_param);
+#else
       av1_iht16x16_256_add(input, dest, stride, txfm_param);
+#endif  // CONFIG_DAALA_DCT16
       break;
     case IDTX: inv_idtx_add_c(input, dest, stride, 16, tx_type); break;
 #endif  // CONFIG_EXT_TX