1. 15 Oct, 2012 6 commits
    • Ronald S. Bultje's avatar
      Merge duplicate loops in tokenization code. · 5cfff7ce
      Ronald S. Bultje authored
      Also merge the three occurrences of 4x4 chroma block writing into a
      single function, and call that function instead of duplicating the
      4x4 chroma tokenization code in 3 places.
      
      Change-Id: I7913538d1029f709b0e3ae49fff1148d3be9eeb9
      5cfff7ce
    • Ronald S. Bultje's avatar
      Minor refactoring in encodeintra.c. · a0ad521c
      Ronald S. Bultje authored
      Merge code blocks for different transform sizes; use MACROBLOCKD as a
      temp variable where that leads to smaller overall source code; remove
      duplicate code under #if CONFIG_HYBRIDTRANSFORM/#else blocks. Some style
      changes to make it follow the style guide a little better.
      
      Change-Id: I1870a06dae298243db46e14c6729c96c66196525
      a0ad521c
    • Ronald S. Bultje's avatar
      Make optimize_mb_16x16() static. · e7a4bcca
      Ronald S. Bultje authored
      It is not used outside this file.
      
      Change-Id: Id0947180edab4c5f1f50589863350b21e97c25f1
      e7a4bcca
    • Ronald S. Bultje's avatar
      Remove assignment in vp8_optimize_mby_16x16(). · 781f6ad7
      Ronald S. Bultje authored
      The result of the assignment is unused.
      
      Change-Id: Id94b790baa6451f24e9aeda6f036390388d93921
      781f6ad7
    • Ronald S. Bultje's avatar
      Remove duplicate code in optimize_mb_NxN vs. vp8_optimize_mby/uv_NxN(). · 11e42d46
      Ronald S. Bultje authored
      Instead, just call vp8_optimize_mby/uv_NxN() inside optimize_mb_NxN().
      
      Change-Id: Ief941064d86db4277ba02e990a26819144b55f7c
      11e42d46
    • Ronald S. Bultje's avatar
      Remove duplicate or unused code in encoder/encodemb.c. · 238ed517
      Ronald S. Bultje authored
      Also make some minor stylistic changes to bring the code closer to
      the style guide. Remove distinction between inter and intra transform
      functions, since both do exactly the same thing except for the check
      against SPLITMV for the second-order transform. Remove some commented
      out debug code. Remove 8x8/16x16 transform code in encode_inter16x16y(),
      since the first-pass only uses 4x4 anyway.
      
      Change-Id: Ife54816ff759825b9141f95dc2ba43c253c14dba
      238ed517
  2. 14 Oct, 2012 6 commits
  3. 12 Oct, 2012 5 commits
  4. 11 Oct, 2012 4 commits
  5. 10 Oct, 2012 4 commits
  6. 09 Oct, 2012 3 commits
    • Scott LaVarnway's avatar
      Removing dr[16] from VP8D_COMP · f56d0679
      Scott LaVarnway authored
      Currently not used.  Also using this as a test commit into the
      experimental branch.
      
      Change-Id: I68c479461ae02f1c7a7f75fa02d1fc0db84bf198
      f56d0679
    • John Koleszar's avatar
      write_kfmodes: use update_skip_probs() · d94d2b75
      John Koleszar authored
      Use the common update_skip_probs() function rather than duplicating its
      logic in write_kf_modes().
      
      Change-Id: I2890a28f6907cb79ffe0fb21d20f0ef98b85cdd9
      d94d2b75
    • Ronald S. Bultje's avatar
      Selectable transform size try 2. · 9ca3c8b2
      Ronald S. Bultje authored
      Results: derf (vanilla or +hybridtx) +0.2% and (+hybrid16x16
      or +tx16x16) +0.7%-0.8%; HD (vanilla or +hybridtx) +0.1-0.2%
      and (+hybrid16x16 or +tx16x16) +1.4%, STD/HD (vanilla or +hybridtx)
      about even, and (+hybrid16x16 or +tx16x16) +0.8-1.0%.
      
      Change-Id: I03899e2f7a64e725a863f32e55366035ba77aa62
      9ca3c8b2
  7. 05 Oct, 2012 3 commits
    • Deb Mukherjee's avatar
    • Deb Mukherjee's avatar
      Mismatch bug-fix when transform expts are off · 105746c8
      Deb Mukherjee authored
      A variety of issues related to tangling of various hybrid transform
      experiments.
      
      Change-Id: I84dac65ff513f8b024407760df55fffa95e198de
      105746c8
    • Paul Wilkins's avatar
      Fix SIMD unsafe use of floating point. · 81c60419
      Paul Wilkins authored
      This commit fixes unsafe simd / floating point interactions arising
      from the current hybrid and 16x16 transform implementation.
      These led to a raft of bugs and issues when the project was
      built using VS2008 for Win32 though they did not show up with
      the unix builds.
      
      Gerrit makes a meal out of presenting the fix but all I have actually
      done is indent the body of each function that uses floating point by
      one level and bracket with emms instructions using  the function
      vp8_clear_system_state(). See below.
      
      function () {
        vp8_clear_system_state();
        {
        ... function body
        }
        vp8_clear_system_state();
      }
      
      This is almost certainly over the top in terms of number of emms
      instructions but is a temporary measure pending implementation of
      integer variants of each function to replace the floating point.
      
      Limited testing suggests that this fixes the problems that arose for
      Win32 VS2008 when the hybrid or 16x16 transforms were enabled.
      
      Change-Id: I7c9a72bd79315246ed880578dec51e2b7c178442
      81c60419
  8. 03 Oct, 2012 1 commit
  9. 27 Sep, 2012 1 commit
    • Deb Mukherjee's avatar
      Entropy coding for hybrid transform · 3f5d60b3
      Deb Mukherjee authored
      Separates the entropy coding context models for 4x4, 8x8 and 16x16
      ADST variants.
      
      There is a small improvement for HD (hd/std-hd) by about 0.1-0.2%.
      Results on derf/yt are about the same, probably because there is not
      enough statistics.
      
      Results may improve somewhat once the initial probability tables are
      updated for the hybrid transforms which is coming soon.
      
      Change-Id: Ic7c0c62dacc68ef551054fdb575be8b8507d32a8
      3f5d60b3
  10. 21 Sep, 2012 1 commit
    • Deb Mukherjee's avatar
      Separating 8x8DCT use from hybridtransform8x8 · 70e21afb
      Deb Mukherjee authored
      With this change, even if hybridtransform8x8 experiment is off,
      8x8 dct is used for the I8x8 mode. However note that the gains
      observed with the hybridtransform8x8 experiment will now be less,
      since part of the gain is now merged in.
      
      Change-Id: I9afb3880906fd0a1368a374041fc08efcf060c54
      70e21afb
  11. 20 Sep, 2012 1 commit
  12. 17 Sep, 2012 1 commit
  13. 14 Sep, 2012 2 commits
  14. 13 Sep, 2012 1 commit
  15. 12 Sep, 2012 1 commit
    • Deb Mukherjee's avatar
      Adds feature for companded MV encoding · 2b26cf17
      Deb Mukherjee authored
      The high-precision (1/8) pel bit is turned off if the reference
      MV is larger than a threshold. The motivation for this patch is
      the intuition that if motion is likely large (as indicated by
      the reference), there is likley to be more motion blur, and as
      a result 1/8 pel precision would be wasteful both in rd sense
      as well as computationally.
      
      The feature is incorporated as part of the newmventropy experiment.
      There is a modest RD improvement with the patch. Overall the
      results with the newmventropy experiment with the threshold being
      16 integer pels are:
      
      derf: +0.279%
      std-hd: +0.617%
      hd: +1.299%
      yt: +0.822%
      
      With threshold 8 integer pels are:
      
      derf: +0.295%
      std-hd: +0.623%
      hd: +1.365%
      yt: +0.847%
      
      Patch: rebased
      Patch: rebase fixes
      
      Change-Id: I4ed14600df3c457944e6541ed407cb6e91fe428b
      2b26cf17