Deb Mukherjee [Mon, 22 Oct 2012 21:43:01 +0000 (14:43 -0700)]
Merging in the Switchable interp experiment
There is a macro DEFAULT_INTERP_FILTER defined in encoder/onyx_if.c that
is set as EIGHTTAP for now - so SWITCHABLE is not really used. Ideally,
this should be SWITCHABLE but that would make the encoder quite a bit slower.
We will change the default filter to SWITCHABLE once we find a faster way to
search for switchable filters.
Change-Id: Iee91832cdc07e6e14108d9b543130fdd12fc9874
Ronald S. Bultje [Tue, 23 Oct 2012 05:04:28 +0000 (22:04 -0700)]
Merge "Change entropy context assignment style from *(x + y) = z to x[y] = z." into experimental
Deb Mukherjee [Mon, 22 Oct 2012 23:52:28 +0000 (16:52 -0700)]
Merging in the TX_SELECT experiment
Change-Id: I0d306b0062be59bc785b65882691c956c8b5e7c8
Deb Mukherjee [Mon, 22 Oct 2012 19:19:00 +0000 (12:19 -0700)]
Merge of the newmventropy experiment
Removes the old mv encoding code.
Change-Id: I1026d48cc5ac15ccb776f98463e929254c3dc7da
Ronald S. Bultje [Mon, 22 Oct 2012 21:06:52 +0000 (14:06 -0700)]
Change entropy context assignment style from *(x + y) = z to x[y] = z.
Change-Id: Ibc4425bb6e01606751021e57c93106c365a2da21
Scott LaVarnway [Mon, 22 Oct 2012 20:57:03 +0000 (13:57 -0700)]
Merge "Added rtcd support vp8_sad16x3 and vp8_sad3x16" into experimental
Scott LaVarnway [Mon, 22 Oct 2012 20:45:42 +0000 (13:45 -0700)]
Added rtcd support vp8_sad16x3 and vp8_sad3x16
Change-Id: I5bca7b7a4b230082d36ac6fb84db84137ad177d7
Ronald S. Bultje [Mon, 22 Oct 2012 19:54:39 +0000 (12:54 -0700)]
Merge changes I02e7f64a,Ide954b00,Idc8b5977 into experimental
* changes:
Fix another typo in 4x4-transform-for-i8x8-intra-pred coeff contexts.
8x8 transform support in splitmv.
Use SPLITMV_PARTITIONING instead of a plain integer type.
Deb Mukherjee [Mon, 22 Oct 2012 19:18:46 +0000 (12:18 -0700)]
Merge "Merging the hybrid transform experiments" into experimental
Deb Mukherjee [Mon, 22 Oct 2012 18:55:29 +0000 (11:55 -0700)]
Merging the hybrid transform experiments
Change-Id: I99f1982b30a630a9a070a8326d83b34a33cba14c
Ronald S. Bultje [Mon, 22 Oct 2012 18:55:06 +0000 (11:55 -0700)]
Fix another typo in 4x4-transform-for-i8x8-intra-pred coeff contexts.
Change-Id: I02e7f64a7c99f6c222b9ef21341b0e801d646810
Ronald S. Bultje [Mon, 22 Oct 2012 18:49:00 +0000 (11:49 -0700)]
8x8 transform support in splitmv.
For splitmv, where partitioning is 8x16, 16x8 or 8x8, this patch
uses the 8x8 transform (instead of the 4x4) if txfm_mode is
ALLOW_8X8 or ALLOW_16X16. For TX_MODE_SELECT, splitmv can indicate
which of the 2 transform sizes (4x4 or 8x8) it wants to use.
Gains (with hybridtx4x4/8x8/16x16 and tx_select experiments
enabled) on derf: +0.9%, HD: +0.4%, STD/HD: +0.8% (SSIM or overall
PSNR, both metrics show similar improvements).
Change-Id: Ide954b000b415548ed92a7ac78e24f36e60fcf06
Scott LaVarnway [Mon, 22 Oct 2012 18:39:08 +0000 (11:39 -0700)]
Merge "Re-enable specialized sse2 block loop filter functions" into experimental
Scott LaVarnway [Mon, 22 Oct 2012 18:38:37 +0000 (11:38 -0700)]
Merge "sse2 intrinsic version of vp8_mbloop_filter_vertical_edge()" into experimental
Scott LaVarnway [Mon, 22 Oct 2012 18:38:21 +0000 (11:38 -0700)]
Merge "sse2 intrinsic version of vp8_mbloop_filter_horizontal_edge()" into experimental
Ronald S. Bultje [Mon, 22 Oct 2012 18:25:48 +0000 (11:25 -0700)]
Use SPLITMV_PARTITIONING instead of a plain integer type.
This can be used to distinguish between 16x8, 8x16, 8x8 and 4x4
partitioning modes when choosing splitmv as a MB mode.
Change-Id: Idc8b59772e1a80ccc4ad44d63c5c2ec3fc061a3c
Ronald S. Bultje [Mon, 22 Oct 2012 18:18:01 +0000 (11:18 -0700)]
Merge "Invert SB probability." into experimental
Jim Bankoski [Mon, 22 Oct 2012 18:12:07 +0000 (11:12 -0700)]
Merge "Remove variance vtable from rtcd" into experimental
Ronald S. Bultje [Mon, 22 Oct 2012 18:05:56 +0000 (11:05 -0700)]
Invert SB probability.
It currently counts the probability that the branch is true, but it
should count the probability that the branch is false.
Change-Id: I963825da2e7a7ed3a613eb23ffd085e427dc36e5
Ronald S. Bultje [Mon, 22 Oct 2012 18:03:25 +0000 (11:03 -0700)]
Merge "Fix use of wrong context in i8x8 coefficient cost calculation." into experimental
Jim Bankoski [Mon, 22 Oct 2012 03:47:57 +0000 (20:47 -0700)]
Remove variance vtable from rtcd
Change-Id: Idd2722a538423b451e1e3495f89a7141480493d6
Deb Mukherjee [Sun, 21 Oct 2012 22:35:24 +0000 (15:35 -0700)]
Merge "Allow B_VL_PRED & B_LD_PRED modes with Superblocks" into experimental
Deb Mukherjee [Fri, 19 Oct 2012 22:12:12 +0000 (15:12 -0700)]
Allow B_VL_PRED & B_LD_PRED modes with Superblocks
Allows B_VL_PRED & B_LD_PRED modes to be used for all blocks
within a MB in B_PRED mode. These modes were temporarily
disabled with super-block coding.
Change-Id: I973b9bdb82c3da5f12d7cc963162a28805f25303
Ronald S. Bultje [Sat, 20 Oct 2012 05:05:50 +0000 (22:05 -0700)]
Fix use of wrong context in i8x8 coefficient cost calculation.
Change-Id: Ica58d0c7d8addb5ddd5d6e1fdea8f0f96e110862
Scott LaVarnway [Fri, 19 Oct 2012 23:02:50 +0000 (16:02 -0700)]
Re-enable specialized sse2 block loop filter functions
vp8_loop_filter_bh and vp8_loop_filter_bv
Change-Id: I0a77d38c701a6cce96cb8371c7f3ab7e2e7f878c
Scott LaVarnway [Fri, 19 Oct 2012 22:52:12 +0000 (15:52 -0700)]
sse2 intrinsic version of vp8_mbloop_filter_vertical_edge()
First sse2 version of vp8_mbloop_filter_vertical_edge(). For now,
intrinsics are being used until the bitstream is finalized. This function
will be revisited later for further performance improvements.
For the test clip used, a 34+% decoder performance improvement
was seen. This will vary depending on material.
Change-Id: I455b438bc8d8af76cf7533ac42eda5f689b21f7c
John Koleszar [Thu, 18 Oct 2012 23:27:30 +0000 (16:27 -0700)]
calculate probs consistently
There were several different methods for calculating bitstream
probabilities in use. Consolodate these into a pair of functions,
get_prob() and get_binary_prob().
Change-Id: I5534f517f74027fee16d89c9baefaafea8156b2f
John Koleszar [Thu, 18 Oct 2012 21:34:53 +0000 (14:34 -0700)]
lint-hunks: better support for working tree
When run with no arguments, report warnings in the diff between the
working tree and HEAD. With arguments, report warnings in the diff
between the named commit and its parents.
Change-Id: Ie10dcdecb303edf8af51bad645cc11206a1fc26b
John Koleszar [Wed, 17 Oct 2012 23:47:38 +0000 (16:47 -0700)]
Remove bc, bc2 from pbi,cpi,xd
Pass the bool coder to be used explicitly. This avoids cases where two
different bool coders can be addressed from the same function. Also be
more consistent with bool coder variable naming, start to standardize
on 'bc'.
Change-Id: I1c95e2fdbe24ebe8c0f84924daa1728e3b054a31
Deb Mukherjee [Mon, 15 Oct 2012 23:41:41 +0000 (16:41 -0700)]
Some cleanups and fixes.
Separates the logic on transform type selection previously spread out
over a number of files into a separate function. Currently the tx_type
field in b_mode_info is not used, but still left in there to eventually
use for signaling the transform type in the bitstream.
Also, now for tx_type = DCT_DCT, the regular integer DCT is used, as
opposed to the floating point DCT used in conjuction with hybrid
transform.
Results change somewhat due to the transform change, but are within
reasonable limits. The hd/std-hd sets are slightly up, while derf/yt
are slightly down.
Change-Id: I5776840c2239ca2da31ca6cfd7fd1148dc5f9e0f
Ronald S. Bultje [Thu, 18 Oct 2012 22:21:30 +0000 (15:21 -0700)]
Merge "Fix encode breakout skip handling." into experimental
Ronald S. Bultje [Thu, 18 Oct 2012 22:21:14 +0000 (15:21 -0700)]
Merge "Detect skippable blocks in splitmv RD loop case." into experimental
Scott LaVarnway [Thu, 18 Oct 2012 21:29:26 +0000 (14:29 -0700)]
sse2 intrinsic version of vp8_mbloop_filter_horizontal_edge()
First sse2 version of vp8_mbloop_filter_horizontal_edge(). For now,
intrinsics are being used until the bitstream is finalized. This function
will be revisited later for further performance improvements.
For the test clip used, a 31+% decoder performance improvement
was seen. This will vary depending on material.
Change-Id: I03ed3a7182478bdd1f094644ff3e0442625600e7
Ronald S. Bultje [Thu, 18 Oct 2012 19:39:10 +0000 (12:39 -0700)]
Merge "Use correct context for optimize_b() in vp8_optimize_mby_8x8()." into experimental
Deb Mukherjee [Thu, 18 Oct 2012 19:00:23 +0000 (12:00 -0700)]
A fix for crashes when #macroblock rows is odd.
Change-Id: If9fc2df4051ccfb8d804dae9d1be08d12d0f81ce
John Koleszar [Thu, 18 Oct 2012 04:43:18 +0000 (21:43 -0700)]
lint-hunks: exit status for only affected lines
Prior to this patch, if there were any lint errors, this script would
exit with an error, even if those errors were not in the hunks being
tested by this script. This change makes it so that if any lint lines
are printed, an error is returned.
Change-Id: I69c8bef4367ccf25d287508f29e587b1f4426143
Ronald S. Bultje [Thu, 18 Oct 2012 01:00:56 +0000 (18:00 -0700)]
Fix encode breakout skip handling.
Change-Id: Ib42a5dbded27abf2a01fd656a47a1e96b1fe2912
John Koleszar [Wed, 17 Oct 2012 23:44:46 +0000 (16:44 -0700)]
Merge "lint-hunks: support operating on arbirary revs" into experimental
Ronald S. Bultje [Wed, 17 Oct 2012 21:32:17 +0000 (14:32 -0700)]
Detect skippable blocks in splitmv RD loop case.
Change-Id: Id8a7265dc721e5cdcaa144e0041beeb32a98f51c
pascal massimino [Wed, 17 Oct 2012 22:17:21 +0000 (15:17 -0700)]
Merge "cosmetic: fix '= & ' construct" into experimental
John Koleszar [Wed, 17 Oct 2012 22:16:20 +0000 (15:16 -0700)]
Merge "rtcd: merge rtcd.c from master" into experimental
John Koleszar [Wed, 17 Oct 2012 22:15:59 +0000 (15:15 -0700)]
Merge "consolidate update_mb_segmentation_map data" into experimental
John Koleszar [Wed, 17 Oct 2012 22:15:10 +0000 (15:15 -0700)]
Merge changes Ic711a378,I43a46c36,If19ee74a into experimental
* changes:
Move remaining per-frame data into partition 0
Interleave modes/residual per macroblock
Force interleaved decoding
John Koleszar [Wed, 17 Oct 2012 21:51:27 +0000 (14:51 -0700)]
cosmetic: fix '= & ' construct
remove useless space after address-of operator.
Change-Id: I1fb9e82e8d6cf87558fbd454fb5c0f87599ca2ab
John Koleszar [Wed, 17 Oct 2012 16:38:13 +0000 (09:38 -0700)]
Move remaining per-frame data into partition 0
This commit moves a bit of data that ended up packed with the
modes/mv/residual partition during the change to interleaved encoding
into partition 0 where it belongs.
Change-Id: Ic711a378c58d9d6a17254384f492c213a15bad92
John Koleszar [Wed, 17 Oct 2012 21:23:08 +0000 (14:23 -0700)]
lint-hunks: support operating on arbirary revs
Rather than diffing only the index, support checking arbitrary commits.
Change-Id: Ia135a487990d8293d1e0799dc062b9f49e020b25
Ronald S. Bultje [Wed, 17 Oct 2012 20:14:56 +0000 (13:14 -0700)]
Use correct context for optimize_b() in vp8_optimize_mby_8x8().
Change-Id: Ic975ab85a9924adc7b3d421f64155cc79f40ffd0
John Koleszar [Tue, 16 Oct 2012 20:52:39 +0000 (13:52 -0700)]
Interleave modes/residual per macroblock
Packs the bitstream with each mb's residual following its mode/mv
information.
TODO: There are still a few fields that should be packed into partition
0 but are included in partition 1, due to them being serialized from
write_kfmodes/pack_inter_mode_mvs, which execute after the first
partition is finalized. These need to be separated out into a separate
function, similar to mb_mode_mv_init() in decodemv.c.
Change-Id: I43a46c363601ab36954d07ebe498760e1e2e3af4
Ronald S. Bultje [Wed, 17 Oct 2012 18:40:00 +0000 (11:40 -0700)]
Use 8x8 block bases for non-4x4 splitmv RD.
Change-Id: I76095d5a02edcc5d404ea830157780b24fd509d7
Scott LaVarnway [Wed, 17 Oct 2012 16:57:23 +0000 (09:57 -0700)]
Merge "Removed the loopfilter rtcd invoke macro code" into experimental
Jim Bankoski [Wed, 17 Oct 2012 16:18:54 +0000 (09:18 -0700)]
removed obselete build dependency
this commit fixes the build on windows with visual studio 2008.
Change-Id: I0baa4044e9e54237da29f2e17332ea6f766dbbec
Scott LaVarnway [Tue, 16 Oct 2012 23:19:35 +0000 (16:19 -0700)]
Removed the loopfilter rtcd invoke macro code
Change-Id: I446b2ffcbe732ffb112dbd97a4799272d4c01a84
John Koleszar [Tue, 16 Oct 2012 21:08:40 +0000 (14:08 -0700)]
Force interleaved decoding
Rather than decoding all modes/mvs separately, decode them per MB. This
forces the mode which was already used form the CONFIG_NEWBESTREFMV and
CONFIG_SUPERBLOCKS experiments, and is a precursor to changing to
interleaved encoding.
Change-Id: If19ee74ac8a987846d1cd0cf2b2e02a82f1a43ad
Jim Bankoski [Tue, 16 Oct 2012 20:49:22 +0000 (13:49 -0700)]
Merge "removed the recon rtcd invoke macro code (unrevert)" into experimental
Ronald S. Bultje [Tue, 16 Oct 2012 19:41:56 +0000 (12:41 -0700)]
Fix a mismatch if a keyframe contains skip-blocks.
Change-Id: I0681d3183f51627be8c2bb76f343b7270f9116d8
Jim Bankoski [Sun, 14 Oct 2012 01:49:44 +0000 (18:49 -0700)]
removed the recon rtcd invoke macro code (unrevert)
This reinstates reverted commit
2113a831575d81faeadd9966e256d58b6b2b1633
Change-Id: I9a9af13497d1e58d4f467e3e083fddf06b1b786c
Ronald S. Bultje [Tue, 16 Oct 2012 02:50:20 +0000 (19:50 -0700)]
Merge "Remove test from B_MODE_INFO." into experimental
Ronald S. Bultje [Tue, 16 Oct 2012 02:50:06 +0000 (19:50 -0700)]
Merge "Remove mode_rdopt from MB_MODE_INFO." into experimental
Ronald S. Bultje [Tue, 16 Oct 2012 02:49:37 +0000 (19:49 -0700)]
Merge "Properly use chroma block object for chroma coefficient costing." into experimental
Yaowu Xu [Mon, 15 Oct 2012 22:02:36 +0000 (15:02 -0700)]
Merge "Changed to use real pixels only for evaluating MVs" into experimental
Yaowu Xu [Mon, 15 Oct 2012 21:30:15 +0000 (14:30 -0700)]
Changed to use real pixels only for evaluating MVs
The commit changed to avoid using pixels from extended border in
in evaluating and select best reference motion vector.
Change-Id: I39b758889373e42ed2889d59744388e5b9c1a20a
Ronald S. Bultje [Mon, 15 Oct 2012 20:54:05 +0000 (13:54 -0700)]
Remove test from B_MODE_INFO.
It is essentially a duplicate of mode for RD-only purposes. Removing it
saves us 4 bytes per B_MODE_INFO, or ~0.5MB for a 1080p video encode.
Change-Id: I0a54db5f51658b3946d7efb1ca6e8cfbda0cdf88
Ronald S. Bultje [Mon, 15 Oct 2012 20:49:45 +0000 (13:49 -0700)]
Remove mode_rdopt from MB_MODE_INFO.
The variable is essentially a duplicate of mode for RD-only purposes.
Removing it gives identical results, and saves 4 bytes per macroblock
(i.e. 32.5kB for a 1080p HD video encode).
Change-Id: I22d5058fdb80ab0b69862caee825e9d86bb148b3
Ronald S. Bultje [Mon, 15 Oct 2012 20:36:00 +0000 (13:36 -0700)]
Properly use chroma block object for chroma coefficient costing.
Change-Id: Ieb38c7aae91dbaca4a8add204fa84e1cfc459933
Ronald S. Bultje [Mon, 15 Oct 2012 17:52:13 +0000 (10:52 -0700)]
Add a new token stuffing function vp8_stuff_mb().
This way a caller doesn't need to implement the logic for which (and how
many) tokens to write out to stuff one macroblock worth of EOBs. Make
the actual function implementations static, since they are now only used
in tokenize.c; also do some minor stylistic changes so it follows the
style guide a little more closely; use PLANE_TYPE where appropriate,
remove old (stale) frame_type function arguments; hardcode plane type
where only a single one is possible (2nd order DC or U/V EOB stuffing);
support stuffing 8x8/4x4 transform EOBs with no 2nd order DC.
Change-Id: Ia448e251d19a4e3182eddeb9edd034bd7dc16fa3
Ronald S. Bultje [Sun, 14 Oct 2012 22:29:56 +0000 (15:29 -0700)]
Add and consistently use PLANE_TYPE.
Change the macros PLANE_TYPE_{Y_NO_DC,Y2,UV,Y_WITH_DC} to a typed enum,
and use this typed enum consistently across all places where relevant.
In places where the type is implied (e.g. in functions that only handle
second order planes or chroma planes), remove it as a function argument
and instead hardcode the proper enum in the code directly.
Change-Id: I93652b4a36aa43163d49c732b0bf5c4442738c47
Ronald S. Bultje [Sat, 13 Oct 2012 18:46:21 +0000 (11:46 -0700)]
Merge duplicate loops in tokenization code.
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
Ronald S. Bultje [Sat, 13 Oct 2012 16:27:54 +0000 (09:27 -0700)]
Minor refactoring in encodeintra.c.
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
Ronald S. Bultje [Sat, 13 Oct 2012 15:31:02 +0000 (08:31 -0700)]
Make optimize_mb_16x16() static.
It is not used outside this file.
Change-Id: Id0947180edab4c5f1f50589863350b21e97c25f1
Ronald S. Bultje [Sat, 13 Oct 2012 15:29:59 +0000 (08:29 -0700)]
Remove assignment in vp8_optimize_mby_16x16().
The result of the assignment is unused.
Change-Id: Id94b790baa6451f24e9aeda6f036390388d93921
Ronald S. Bultje [Sat, 13 Oct 2012 15:26:05 +0000 (08:26 -0700)]
Remove duplicate code in optimize_mb_NxN vs. vp8_optimize_mby/uv_NxN().
Instead, just call vp8_optimize_mby/uv_NxN() inside optimize_mb_NxN().
Change-Id: Ief941064d86db4277ba02e990a26819144b55f7c
Ronald S. Bultje [Sat, 13 Oct 2012 15:15:51 +0000 (08:15 -0700)]
Remove duplicate or unused code in encoder/encodemb.c.
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
Ronald S. Bultje [Sat, 13 Oct 2012 05:42:06 +0000 (22:42 -0700)]
Remove duplicate or unused code in encoder/quantize.c.
Also make some minor stylistic changes to bring the code closer to
the style guide. Remove checks against i8x8/bpred in the mb-codepath,
since these do individual block reconstruction and thus don't go through
this codepath.
Change-Id: I4dfcf8f78746f4647a206475acf731837aa4fd47
Ronald S. Bultje [Sat, 13 Oct 2012 04:59:50 +0000 (21:59 -0700)]
Remove duplicate or unused code in common/invtrans.c.
Also make some minor stylistic changes to bring the code closer to
the style guide.
Change-Id: Idafa280724f49868ed6b7d65b7fd9227d781994b
Ronald S. Bultje [Sat, 13 Oct 2012 04:41:58 +0000 (21:41 -0700)]
Add a _4x4 sufix to all 4x4-transform-related functions.
This includes trellis optimization, forward/inverse transform,
quantization, tokenization and stuffing functions.
Change-Id: Ibd34132e1bf0cd667671a57b3f25b3d361b9bf8a
Ronald S. Bultje [Sat, 13 Oct 2012 04:08:03 +0000 (21:08 -0700)]
Replace vp8_inverse_htransform_b() by direct call to vp8_ihtllm_c().
Change-Id: I55ee7566341b04d7e7340fdab95189f82231a792
Jim Bankoski [Sun, 14 Oct 2012 03:29:04 +0000 (20:29 -0700)]
Revert "removed the recon. rtcd invoke macro code"
This reverts commit
2113a831575d81faeadd9966e256d58b6b2b1633
Jim Bankoski [Sun, 14 Oct 2012 01:49:44 +0000 (18:49 -0700)]
removed the recon. rtcd invoke macro code
Code clean up - removed rtcd
Change-Id: Id963ecf53c370b1d99484ef18d6befeed7e0c748
Adrian Grange [Fri, 12 Oct 2012 22:08:24 +0000 (15:08 -0700)]
Merge "Remove reverting of tx-select if only a single txfm-size is used." into experimental
Yaowu Xu [Fri, 12 Oct 2012 18:38:10 +0000 (11:38 -0700)]
removed the dependency cross key frame boundary
This allowed decoding from any key frame after skipping the frames
prior.
Change-Id: I096fccad5346d75ae50a017c94eb8d772a7e8b00
Yaowu Xu [Fri, 12 Oct 2012 19:23:18 +0000 (12:23 -0700)]
Merge "added the ablility to skip first n frames for vpxdec" into experimental
Ronald S. Bultje [Fri, 12 Oct 2012 01:19:20 +0000 (18:19 -0700)]
Remove reverting of tx-select if only a single txfm-size is used.
Entropy coding takes care of this anyway, and this causes changes to
the txfm size assigned to skip blocks, which can affect the loopfilter
output, thus causing encoder/decoding mismatches.
Change-Id: I591a8d8a4758a507986b751a9f83e6d76e406998
Ronald S. Bultje [Fri, 12 Oct 2012 18:06:55 +0000 (11:06 -0700)]
Remove CONFIG_TX16X16, since the experiment was merged.
Change-Id: Ib74824c824f7ebb279e7df8eb32440d4d65fbd79
John Koleszar [Fri, 12 Oct 2012 05:15:33 +0000 (22:15 -0700)]
consolidate update_mb_segmentation_map data
The update_mb_segmentation_map flag was being signalled earlier than
other data dependent on that flag. Consolidate this data so it's
parsed within the same if-scope as the flag is originally parsed in.
Change-Id: I10e90b4f511856445ef75a85a44ff441e1e5e672
Yaowu Xu [Thu, 11 Oct 2012 22:12:35 +0000 (15:12 -0700)]
added the ablility to skip first n frames for vpxdec
Change-Id: Id7d1f25078be92dd6cda246738ad6fb50579f555
Ronald S. Bultje [Thu, 11 Oct 2012 17:13:48 +0000 (10:13 -0700)]
Add encoder/decoder mismatch information to internal stats file.
Change-Id: Ibb6ba26e8718d3af27553ca59443a8c6aec7749d
Ronald S. Bultje [Thu, 11 Oct 2012 00:18:22 +0000 (17:18 -0700)]
Use transform-size selection for i8x8_pred also.
Change-Id: Iecb282fc89f9b5145ef31c5eda294ad42bc32a5d
Deb Mukherjee [Wed, 10 Oct 2012 18:20:33 +0000 (11:20 -0700)]
Merge of the TX_16X16 experiment
Change-Id: I22aa803ffff330622cdb77277e7b196a9766f882
John Koleszar [Wed, 10 Oct 2012 19:30:16 +0000 (12:30 -0700)]
rtcd: merge rtcd.c from master
Pick up changes to multithreading and once() implementations.
Change-Id: I4a98df6b714f3208643798e0e7f74e373a7931e3
Scott LaVarnway [Wed, 10 Oct 2012 16:08:03 +0000 (09:08 -0700)]
Merge "Removing dr[16] from VP8D_COMP" into experimental
John Koleszar [Wed, 10 Oct 2012 15:49:50 +0000 (08:49 -0700)]
Merge "write_kfmodes: use update_skip_probs()" into experimental
Paul Wilkins [Wed, 10 Oct 2012 12:45:19 +0000 (05:45 -0700)]
Merge "convert copy16x16 to rtcd" into experimental
Jim Bankoski [Wed, 10 Oct 2012 00:09:08 +0000 (17:09 -0700)]
convert copy16x16 to rtcd
Convert copy16x16 from invoke to rtcd. The first in a long
string of converts.
Change-Id: I296b0aa32f40e9fb649f7a3cb914a4e5300cad63
Scott LaVarnway [Tue, 9 Oct 2012 22:54:32 +0000 (15:54 -0700)]
Removing dr[16] from VP8D_COMP
Currently not used. Also using this as a test commit into the
experimental branch.
Change-Id: I68c479461ae02f1c7a7f75fa02d1fc0db84bf198
John Koleszar [Tue, 9 Oct 2012 22:35:57 +0000 (15:35 -0700)]
write_kfmodes: use update_skip_probs()
Use the common update_skip_probs() function rather than duplicating its
logic in write_kf_modes().
Change-Id: I2890a28f6907cb79ffe0fb21d20f0ef98b85cdd9
Ronald S. Bultje [Tue, 9 Oct 2012 16:18:21 +0000 (09:18 -0700)]
Selectable transform size try 2.
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
Deb Mukherjee [Fri, 5 Oct 2012 13:49:08 +0000 (06:49 -0700)]
Merge "Mismatch bug-fix when transform expts are off" into experimental
Deb Mukherjee [Fri, 5 Oct 2012 12:08:20 +0000 (05:08 -0700)]
Mismatch bug-fix when transform expts are off
A variety of issues related to tangling of various hybrid transform
experiments.
Change-Id: I84dac65ff513f8b024407760df55fffa95e198de
Paul Wilkins [Fri, 5 Oct 2012 10:16:46 +0000 (11:16 +0100)]
Fix SIMD unsafe use of floating point.
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
John Koleszar [Wed, 3 Oct 2012 15:28:51 +0000 (08:28 -0700)]
Merge "all_builds.py: add ability to pass extra configure flags" into experimental