John Koleszar [Tue, 22 Jun 2010 12:44:48 +0000 (08:44 -0400)]
msvs: fix install of codec sources
The libs.mk file must be installed for the vpx.vcproj file to be
generated. It was being installed, but not in the src/ directory as
expected.
Also missed include files yasm.rules, quantize_x86.h
Change-Id: Ic1a6f836e953bfc954d6e42a18c102a0114821eb
Tom Finegan [Thu, 22 Jul 2010 21:51:17 +0000 (17:51 -0400)]
Change devenv.com command line.
Change /build to -build to avoid problems when builds are run within
msys bash shells.
Change-Id: Ie68d72f702adad00d99be8a01c7a388c3af7657d
Tom Finegan [Thu, 22 Jul 2010 17:34:25 +0000 (13:34 -0400)]
Add vs9 targets.
Add targets x86-win32-vs9 and x86_64-win64-vs9 for support of Visual
Studio 2008-- this removes the need to convert the vs8 projects before
using them within the IDE.
Change-Id: Idb83e2ae701e07d98db1be71638280a493d770a2
Johann [Wed, 21 Jul 2010 19:59:39 +0000 (12:59 -0700)]
Merge "limit range checking code for L[k] to CONFIG_DEBUG. patch by timeless@gmail.com"
Yaowu Xu [Mon, 19 Jul 2010 14:50:26 +0000 (07:50 -0700)]
Merge "Improve the accuracy of forward walsh-hadamard transform"
Paul Wilkins [Mon, 19 Jul 2010 12:28:34 +0000 (13:28 +0100)]
ARNR Lookup Table.
Change submitted for Adrian Grange. Convert threshold
calculation in ARNR filter to a lookup table.
Change-Id: I12a4bbb96b9ce6231ce2a6ecc2d295610d49e7ec
Paul Wilkins [Mon, 19 Jul 2010 10:32:09 +0000 (11:32 +0100)]
Parameter limit change.
Change maximum ARNR filter width to 15.
Change-Id: I3b72450ea08e96287445ec18810630ee2292954c
Paul Wilkins [Mon, 19 Jul 2010 13:10:07 +0000 (14:10 +0100)]
Rate control fix for ARNR filtered frames.
Previously we had assumed that it was necessary to give a full frame's
bit allocation to the alt ref frame if it has been created through temporal
filtering. This is not the case. The active max quantizer control
insures that sufficient bits are allocated if needed and allocating a
full frame's worth of bits creates an excessive overhead for the ARF.
Change-Id: I83c95ed7bc7ce0e53ccae6ff32db5a97f145937a
Paul Wilkins [Fri, 16 Jul 2010 14:57:17 +0000 (15:57 +0100)]
Fix: Incorrect 'cols' calculation in temporal filter.
Change-Id: I37f10fbe4fbb505c1d34980a59af3e817c287e22
Michael Kohler [Mon, 12 Jul 2010 16:41:45 +0000 (18:41 +0200)]
limit range checking code for L[k] to CONFIG_DEBUG. patch by timeless@gmail.com
John Koleszar [Tue, 22 Jun 2010 13:45:43 +0000 (09:45 -0400)]
msvs: disable CRT deprecation warnings
Disables the warnings produced for so-called insecure standard C
functions.
Change-Id: I0e6f448e27f899a0eaefc1151185945fbe15718e
John Koleszar [Wed, 7 Jul 2010 23:57:37 +0000 (16:57 -0700)]
Merge "Fix misspelled "paramter" in vpx_codec_internal.h" to "parameter"."
John Koleszar [Wed, 7 Jul 2010 23:57:08 +0000 (16:57 -0700)]
Merge "Fix misspelled "skiped" in onyxc_int.h to "skipped"."
Yaowu Xu [Wed, 7 Jul 2010 17:26:30 +0000 (10:26 -0700)]
Fix a compiling error on armv6
The issue was caused by a bad merge in Change I5559d1e8
Change-Id: I6563f652bc1500202de361f8f51d11cc6ddf3331
Michael Kohler [Wed, 7 Jul 2010 17:49:58 +0000 (19:49 +0200)]
Fix misspelled "paramter" in vpx_codec_internal.h" to "parameter".
Signed-off-by: Michael Kohler <michaelkohler@live.com>
Michael Kohler [Wed, 7 Jul 2010 17:48:12 +0000 (19:48 +0200)]
Fix misspelled "skiped" in onyxc_int.h to "skipped".
Signed-off-by: Michael Kohler <michaelkohler@live.com>
Adrian Grange [Thu, 1 Jul 2010 13:17:04 +0000 (14:17 +0100)]
Fix bug in 1st pass motion compensation
In the case where the best reference mv is not (0,0) a secondary
search is carried out centered on (0,0). However, rather than
sending tmp_err into the search function, motion_error was
inadvertently passed.
As a result tmp_err remains set at INT_MAX and the (0,0)-based
search result will never be selected, even if it is better.
Change-Id: I3c82b246c8c82ba887b9d3fb4c9e0a0f2fe5a76c
Paul Wilkins [Thu, 1 Jul 2010 08:53:40 +0000 (01:53 -0700)]
Merge "Further adjustment of RD behaviour with Q and Zbin."
John Koleszar [Thu, 1 Jul 2010 00:19:26 +0000 (17:19 -0700)]
Merge "ivfenc: correct fixed kf interval, --disable-kf"
John Koleszar [Wed, 30 Jun 2010 15:01:02 +0000 (08:01 -0700)]
Merge "ARM WinCE VS8 build update"
John Koleszar [Wed, 30 Jun 2010 14:59:39 +0000 (07:59 -0700)]
Merge "Remove INLINE/FORCEINLINE"
John Koleszar [Wed, 30 Jun 2010 14:22:40 +0000 (10:22 -0400)]
Update loopfilter frame/filter/sharp info for multithread
Change I9fd1a5a4 updated the multithreaded loopfilter to avoid
reinitializing several parameteres if they haven't changed from the
last frame, but the code to update the last frame's parameters wasn't
invoked in the multithreaded case.
Change-Id: Ia23d937af625c01dd739608e02d110f742b7e1f2
Yunqing Wang [Wed, 30 Jun 2010 13:56:36 +0000 (06:56 -0700)]
Merge "Add loopfilter initialization fix in multithreading code"
Yunqing Wang [Wed, 30 Jun 2010 13:42:39 +0000 (09:42 -0400)]
Add loopfilter initialization fix in multithreading code
Modified loopfilter initialization to avoid unnecessary operations.
Change-Id: I9fd1a5a49edc1cb8116c2a72a6908b1e437459ec
Adrian Grange [Wed, 30 Jun 2010 09:43:47 +0000 (02:43 -0700)]
Merge "Fixed buffer selection for UV in AltRef filtering"
Yunqing Wang [Fri, 25 Jun 2010 13:18:11 +0000 (09:18 -0400)]
Improve SSE2 loopfilter functions
Restructured and rewrote SSE2 loopfilter functions. Combined u and
v into one function to take advantage of SSE2 128-bit registers.
Tests on test clips showed a 4% decoder performance improvement on
Linux desktop.
Change-Id: Iccc6669f09e17f2224da715f7547d6f93b0a4987
James Zern [Tue, 29 Jun 2010 16:02:19 +0000 (12:02 -0400)]
ARM WinCE VS8 build update
The generated project is vpx.vcproj, change vpx_decoder references to
match. Remove .rules file dependency as it will be pulled from the
source tree.
Change-Id: I679db2748b37adae3bafd764dba8575fc3abde72
Paul Wilkins [Tue, 29 Jun 2010 11:15:54 +0000 (12:15 +0100)]
Further adjustment of RD behaviour with Q and Zbin.
Following conversations with Tim T (Derf) I ran a large number of
tests comparing the existing polynomial expression with a simpler
^2 variant. Though the polynomial was sometimes a little better at
the extremes of Q it was possible to get close for most clips and
even a little better on some.
This code also changes the way the RD multiplier is calculated
when the ZBIN is extended to use a variant of the same ^2
expression.
I hope that this simpler expression will be easier to tune further
as we expand our test set and consider adjustments based on content.
Change-Id: I73b2564346e74d1332c33e2c1964ae093437456c
Yaowu Xu [Tue, 29 Jun 2010 05:03:43 +0000 (22:03 -0700)]
Improve the accuracy of forward walsh-hadamard transform
Besides the slight improvement in round trip error. This
also fixes a sign bias in the forward transform, so the
round trip errors are evenly distributed between +1s and
-1s. The old bias seemed to work well with the dc sign bias
in old fdct, which no longer exist in the improved fdct.
Change-Id: I8635e7be16c69e69a8669eca5438550d23089cef
Adrian Grange [Mon, 28 Jun 2010 11:00:11 +0000 (12:00 +0100)]
Fixed buffer selection for UV in AltRef filtering
Corrected setting of "which_buffer" for U & V cases to match that
used for Y, i.e. to refer to the temporally most recent frame of
those to be filtered.
Change-Id: Idf94b287ef47a05f060da3e61134a0b616adcb6b
Scott LaVarnway [Thu, 24 Jun 2010 17:11:30 +0000 (13:11 -0400)]
Added first-pass sse2 version of Yaowu's new fdct.
Change-Id: Ib479210067510162879c368428b92690591120b2
Yaowu Xu [Wed, 16 Jun 2010 19:52:18 +0000 (12:52 -0700)]
Redo the forward 4x4 dct
The new fdct lowers the round trip sum squared error for a
4x4 block ~0.12. or ~0.008/pixel. For reference, the old
matrix multiply version has average round trip error 1.46
for a 4x4 block.
Thanks to "derf" for his suggestions and references.
Change-Id: I5559d1e81d333b319404ab16b336b739f87afc79
Fritz Koenig [Thu, 24 Jun 2010 18:30:48 +0000 (14:30 -0400)]
vp8cx : bestsad declared and initialized incorrectly.
bestsad needs to be a int and set to INT_MAX because at the end
of the function it is compared to INT_MAX to determine if there
was a match in the function.
Change-Id: Ie80e88e4c4bb4a1ff9446079b794d14d5a219788
Fritz Koenig [Thu, 24 Jun 2010 16:18:23 +0000 (12:18 -0400)]
vp8cx : bestsad declared and initialized incorrectly.
bestsad should be an int initialized to INT_MAX. The optimized
SAD function expects a signed value for bestsad to use for comparison
and early loop termination. When no match is made, which is
determined by a comparison of bestsad to INT_MAX, INT_MAX is returned.
John Koleszar [Thu, 24 Jun 2010 13:02:48 +0000 (09:02 -0400)]
Remove INLINE/FORCEINLINE
These are mostly vestigial, it's up to the compiler to decide what
should be inlined, and this collided with certain Windows platform SDKs.
Change-Id: I80dd35de25eda7773156e355b5aef8f7e44e179b
John Koleszar [Thu, 24 Jun 2010 12:51:00 +0000 (08:51 -0400)]
configure: remove postproc-generic
This option is vestigial and is unreferenced.
Change-Id: I8bd27cb674c263e9a86fb43244003a9b9df3ca9c
John Koleszar [Thu, 24 Jun 2010 12:41:51 +0000 (08:41 -0400)]
ivfenc: correct fixed kf interval, --disable-kf
ivfenc was setting the VPX_KF_FIXED mode when the kf_min_dist and
kf_max_dist parameters were set to each other. This flag actually means
that keyframes are disabled, and that name was deprecated to avoid
confusion such as this. Instead, a new option is exposed for setting the
VPX_KF_DISABLED mode, and the intervals are passed through to the codec,
which will do automatic placement at a fixed interval as expected.
Change-Id: I15abbec5936f39d5901878b4bc154372fbc23a43
John Koleszar [Mon, 21 Jun 2010 17:00:36 +0000 (10:00 -0700)]
Merge "Remove deprecated `svnstat' rule from Makefile"
agrange [Mon, 21 Jun 2010 12:44:42 +0000 (13:44 +0100)]
Fix breakout thresh computation for golden & AltRef frames
1. Unavailability of each reference frame type should be tested
independently,
2. Also, only the VP8_GOLD_FLAG needs to be tested before setting
golden frame specific thresholds, and only VP8_ALT_FLAG needs
testing before setting thresholds relevant to the AltRef frame.
(Raised by gbvalor, in response to Issue 47)
Change-Id: I6a06fc2a6592841d85422bc1661e33349bb6c3b8
Giuseppe Scrivano [Mon, 21 Jun 2010 15:15:04 +0000 (17:15 +0200)]
Remove deprecated `svnstat' rule from Makefile
agrange [Fri, 18 Jun 2010 14:58:18 +0000 (15:58 +0100)]
Changed unary operator from ! to ~
Since the intent is
to reset the appropriate bit in ref_frame_flags not to
test a logic condition. Prior result would always have
been ref_frame_flags being set to 0.
(Issue reported by dgohman, issue 47)
Change-Id: I2c12502ed74c73cf38e98c9680e0249c29e16433
agrange [Fri, 18 Jun 2010 14:18:09 +0000 (15:18 +0100)]
Moved DOUBLE_DIVIDE_CHECK to denominator (was on numerator)
The DOUBLE_DIVIDE_CHECK macro prevents from divide by 0,
so must be on the denominator to work as intended.
Change-Id: Ie109242d52dbb9a2c4bc1e11890fa51b5f87ffc7
Timothy B. Terriberry [Fri, 18 Jun 2010 02:33:52 +0000 (19:33 -0700)]
Fix a linker error on x86-64 Linux when not using a version script.
If the version script produced by the libvpx build system is not
used when linking a shared library on x86-64 Linux, the constant
data in the subpel filters produces R_X86_64_32 relocation errors
due to the use of wrt rip addressing instead of
wrt rip wrt ..gotpcrel.
Instead of adding a new macro for this addressing mode, this patch
sets the ELF visibility of these symbols to "hidden", which
allows wrt rip addressing to work without a text relocation.
This allows building a shared library without using the provided
build system or a separate version script.
Fixes http://code.google.com/p/webm/issues/detail?id=46
Change-Id: Ie108f9d9a4352e5af46938bf4750d2302c1b2dc2
Jim Bankoski [Wed, 16 Jun 2010 16:36:53 +0000 (12:36 -0400)]
vp8_block_error_xmm: remove unnecessary instructions
Remove a couple instructions from this function which weren't
necessary for correct execution.
Change-Id: Ib649674f140689f7e5c1530c35686241688a3151
John Koleszar [Fri, 18 Jun 2010 17:26:16 +0000 (10:26 -0700)]
Merge "cosmetics: trim trailing whitespace"
Yunqing Wang [Fri, 18 Jun 2010 17:25:26 +0000 (10:25 -0700)]
Merge "Add x86_64-linux-icc target to build VP8 with icc"
Yunqing Wang [Thu, 17 Jun 2010 17:34:19 +0000 (13:34 -0400)]
Add x86_64-linux-icc target to build VP8 with icc
Add a target for icc.
Change-Id: Ia1db82373d9c7268848bbb65c9483d408b9d933f
John Koleszar [Fri, 18 Jun 2010 16:39:21 +0000 (12:39 -0400)]
cosmetics: trim trailing whitespace
When the license headers were updated, they accidentally contained
trailing whitespace, so unfortunately we have to touch all the files
again.
Change-Id: I236c05fade06589e417179c0444cb39b09e4200d
John Koleszar [Fri, 18 Jun 2010 01:08:36 +0000 (18:08 -0700)]
Merge "Change bitreader to use a larger window."
John Koleszar [Thu, 17 Jun 2010 13:07:33 +0000 (09:07 -0400)]
CHANGELOG: 0.9.1
Change-Id: Icca54b9d51becc49255193801762e1936a07aa2d
John Koleszar [Wed, 16 Jun 2010 19:44:07 +0000 (15:44 -0400)]
Update AUTHORS
Change-Id: I1c6a275278788dfdc630ed436d2c770acfcbd097
John Koleszar [Wed, 16 Jun 2010 19:43:09 +0000 (15:43 -0400)]
Generate AUTHORS file with a script
This information is in git, so it's better to use that as a source than
updating this file manually. This script can be run manually at release
time for now, or we can set up a cron job sometime in the future.
Change-Id: I0344135ceb9c04ed14e2e2d939a93194e35973db
John Koleszar [Wed, 16 Jun 2010 18:54:39 +0000 (11:54 -0700)]
Merge "Avoid encoding garbage when ivfenc encounters an unsupported Y4M file."
John Koleszar [Wed, 16 Jun 2010 18:54:10 +0000 (11:54 -0700)]
Merge "gen_scalers: fix 64-bit integer promotion bug"
Tom Finegan [Wed, 16 Jun 2010 17:24:55 +0000 (13:24 -0400)]
Avoid encoding garbage when ivfenc encounters an unsupported Y4M file.
This change stops ivfenc from treating unsupported Y4M files as raw
input.
For example, if given an interlaced Y4M file, ivfenc treated the input
as if it were raw data because the unsupported Y4M file case previously
fell through without being handled.
Change-Id: I06caa50f3448e6388741a77346daaebf77c277e1
John Koleszar [Wed, 16 Jun 2010 16:27:52 +0000 (12:27 -0400)]
gen_scalers: fix 64-bit integer promotion bug
i needs to be treated as signed to get the proper indexing on 64-bit
platforms. This behavior was accidentally reverted when fixing an
unsigned/signed comparison warning.
Change-Id: Ic306d609bdc8de94c8f8ba29c6e45c736101a82e
Timothy B. Terriberry [Wed, 5 May 2010 21:58:19 +0000 (17:58 -0400)]
Change bitreader to use a larger window.
Change bitreading functions to use a larger window which is refilled less
often.
This makes it cheap enough to do bounds checking each time the window is
refilled, which avoids the need to copy the input into a large circular
buffer.
This uses less memory and speeds up the total decode time by 1.6% on an ARM11,
2.8% on a Cortex A8, and 2.2% on x86-32, but less than 1% on x86-64.
Inlining vp8dx_bool_decoder_fill() has a big penalty on x86-32, as does moving
the refill loop to the front of vp8dx_decode_bool().
However, having the refill loop between computation of the split values and
the branch in vp8_decode_mb_tokens() is a big win on ARM (presumably due to
memory latency and code size: refilling after normalization duplicates the
code in the DECODE_AND_BRANCH_IF_ZERO and DECODE_AND_LOOP_IF_ZERO cases.
Unfortunately, refilling at the end of vp8dx_bool_decoder_fill() and at the
beginning of each decode step in vp8_decode_mb_tokens() means the latter
requires an extra refill at the end.
Platform-specific versions could avoid the problem, but would require most of
detokenize.c to be duplicated.
Change-Id: I16c782a63376f2a15b78f8086d899b987204c1c7
Yunqing Wang [Tue, 15 Jun 2010 13:41:54 +0000 (06:41 -0700)]
Merge "More on "some XMM registers are non-volatile on windows x64 ABI""
Yunqing Wang [Tue, 15 Jun 2010 13:11:26 +0000 (09:11 -0400)]
More on "some XMM registers are non-volatile on windows x64 ABI"
Add same fix in subpixel_sse2.asm.
Change-Id: Icfda6103cbf74ec43308e96961dd738aa823c14d
James Zern [Fri, 4 Jun 2010 22:05:12 +0000 (18:05 -0400)]
VisualStudio projects: asm tool updates
vs8
- pull yasm.rules [1] into the source tree to avoid need to install
file into VC/VCProjectDefaults
- reference same w/ToolFile & RelativePath
- update arm branch to match
vs7:
- quote source file paths passed to yasm
[1]:
http://www.tortall.net/svn/yasm/trunk/yasm/Mkfiles/vc9/yasm.rules@2271
Change-Id: I52b801496340cd7b1d0023d12afbc04624ecefc3
John Koleszar [Mon, 14 Jun 2010 14:22:55 +0000 (10:22 -0400)]
vp8_cx_iface: set default cpu used to 0
Change-Id: I7b35f4717cdd204224112f72471b551617262417
Guillermo Ballester Valor [Fri, 11 Jun 2010 18:33:49 +0000 (14:33 -0400)]
Fix compiler warnings
Change-Id: I2a97f08cc3c7808ce5be39e910cc5147ecf03a1d
Scott LaVarnway [Mon, 14 Jun 2010 18:07:56 +0000 (14:07 -0400)]
sse2 version of vp8_regular_quantize_b
Added sse2 version of vp8_regular_quantize_b which improved encode
performance(for the clip used) by ~10% for 32 bit builds and ~3% for
64 bit builds.
Also updated SHADOW_ARGS_TO_STACK to allow for more than 9 arguments.
Change-Id: I62f78eabc8040b39f3ffdf21be175811e96b39af
Paul Wilkins [Mon, 14 Jun 2010 16:55:02 +0000 (09:55 -0700)]
Merge "Use local pointer to pbi->common."
John Koleszar [Mon, 14 Jun 2010 13:09:47 +0000 (06:09 -0700)]
Merge "platform autodetect: accept amd64 as a synonym for x86_64"
Fabio Pedretti [Mon, 14 Jun 2010 13:05:35 +0000 (09:05 -0400)]
Remove useless 500 frame limit
Change-Id: Ib82de60cf32cf08844c3e2d88d7c587396f3892c
John Koleszar [Mon, 14 Jun 2010 12:54:47 +0000 (05:54 -0700)]
Merge "ivfenc: fix two-pass support of raw files"
Andres Mejia [Mon, 14 Jun 2010 05:27:33 +0000 (01:27 -0400)]
Use public domain implementation for MD5 algorithm
The RSA Data Security, Inc. implementation license bears a requirement
similar to the old problematic BSD license with advertising clause.
Change-Id: I877b71ff0548934b1c4fd87245696f53dedbdf26
John Koleszar [Mon, 14 Jun 2010 12:33:54 +0000 (08:33 -0400)]
ivfenc: fix two-pass support of raw files
Commit 3245d46 "ivfenc: support reading/writing from a pipe" broke
support for two pass encodes of raw files when done in two
invocations of ivfenc. The raw image was only set up on pass 0,
which was never hit when running with --pass=2 --passes=2.
Change-Id: I6a9858be1a8998d5bd45331123b46b1baa05b379
John Koleszar [Sun, 13 Jun 2010 21:35:21 +0000 (14:35 -0700)]
Merge "Make this/next iiratio unsigned."
Paul Wilkins [Sun, 13 Jun 2010 11:01:46 +0000 (04:01 -0700)]
Merge "Tuning of baseline Rd equation to improve behavior at the"
Paul Wilkins [Sun, 13 Jun 2010 11:01:01 +0000 (04:01 -0700)]
Merge "Incorrect comment."
John Koleszar [Sat, 12 Jun 2010 18:11:51 +0000 (14:11 -0400)]
Make this/next iiratio unsigned.
This patch addresses issue #79, which is a regression since commit
28de670 "Fix RD bug." If the coded error value is zero, the iiratio
calculation effectively multiplies by 1000000 by the
DOUBLE_DIVIDE_CHECK macro. This can result in a value larger than
INT_MAX, giving a negative ratio. Since the error values are
conceptually unsigned (though they're stored in a double) this patch
makes the iiratio values unsigned, which allows the clamping to work
as expected.
John Koleszar [Sat, 12 Jun 2010 14:10:39 +0000 (07:10 -0700)]
Merge "require --enable-psnr to build ssim"
John Koleszar [Thu, 10 Jun 2010 21:10:12 +0000 (17:10 -0400)]
ivfenc: support reading/writing from a pipe
Use - for the filename to use stdin/stdout. Update to avoid opening
the file multiple times.
Change-Id: I356356fa16bb334d4b22abc531dc03c0d95917a3
John Koleszar [Fri, 11 Jun 2010 18:36:38 +0000 (11:36 -0700)]
Merge "Enable vp8_sad16x16x4d_sse3 in non-RTCD case"
Yaowu Xu [Fri, 11 Jun 2010 18:14:25 +0000 (11:14 -0700)]
Merge "Change preprocessor check to _WIN32"
John Koleszar [Fri, 11 Jun 2010 17:15:30 +0000 (13:15 -0400)]
Enable vp8_sad16x16x4d_sse3 in non-RTCD case
Typo caused C version of 16x16x4 SAD to be called when built with
--disable-runtime-cpu-detect.
Change-Id: I0fe6fa67280b3a5f13acb3c8ed914f039aaaf316
John Koleszar [Fri, 11 Jun 2010 17:05:08 +0000 (13:05 -0400)]
require --enable-psnr to build ssim
ssim.c comiles in a huge (512M) amount of global scratch space. Allocating
this data on the heap would be a better solution, but this file doesn't
need to be built at all in most cases, so as a first pass, disable it
except when doing opsnr.stt output (--enable-psnr).
Change-Id: I320d812f6d652a12516a16b52295ebff20b5bd42
John Koleszar [Fri, 11 Jun 2010 16:16:36 +0000 (12:16 -0400)]
platform autodetect: accept amd64 as a synonym for x86_64
Thanks to James Cloos <cloos at jhcloos dot com> for the tip.
Change-Id: If377cc084dd7c16a4f51191a2aa0d83e7117ebec
Makoto Kato [Fri, 11 Jun 2010 09:32:28 +0000 (18:32 +0900)]
some XMM registers are non-volatile on windows x64 ABI
XMM6 to XMM15 are non-volatile on Windows x64 ABI. We have to save
these registers.
Change-Id: I4676309f1350af25c8a35f0c81b1f0499ab99076
Paul Wilkins [Fri, 11 Jun 2010 15:12:45 +0000 (16:12 +0100)]
Incorrect comment.
(Thanks to Ronald S. Bultje)
Paul Wilkins [Fri, 11 Jun 2010 14:17:57 +0000 (15:17 +0100)]
Use local pointer to pbi->common.
Paul Wilkins [Fri, 11 Jun 2010 14:10:51 +0000 (15:10 +0100)]
Tuning of baseline Rd equation to improve behavior at the
low and high Q ends.
Frank Galligan [Thu, 10 Jun 2010 20:59:21 +0000 (16:59 -0400)]
Change preprocessor check to _WIN32
Change-Id: I841dc0b8ebb150ac998f4076c148d7bb187e4301
Yunqing Wang [Fri, 11 Jun 2010 13:48:52 +0000 (06:48 -0700)]
Merge "Improve vp8_sixtap_predict functions"
John Koleszar [Thu, 10 Jun 2010 12:56:31 +0000 (08:56 -0400)]
replace while(0) construct with if/else
No good reason to be tricky here. I don't know why 'break' occurred to me
as the natrual replacement for the 'return', but an if/else block is
definitely clearer.
Change-Id: I08a336307afeb0dc7efa494b37398f239f66c2cf
Timothy B. Terriberry [Thu, 10 Jun 2010 22:42:24 +0000 (18:42 -0400)]
Fix new MV clamping scheme for chroma MVs.
The new scheme introduced in I68d35a2f did not clamp chroma MVs in the SPLITMV
case, and clamped them incorrectly (to the luma plane bounds) in every other
case.
Because chroma MVs are computed from the luma MVs before clamping occurs, they
could still point outside of the frame buffer and cause crashes.
This clamping happens outside of the MV prediction loop, and so should not
affect bitstream decoding.
John Koleszar [Thu, 10 Jun 2010 16:08:01 +0000 (12:08 -0400)]
Remove reference to 'vpx Technologies'
Vestigial.
Change-Id: Iffa9e6d5ba5199b136d7549890101da17c11e3c3
John Koleszar [Thu, 10 Jun 2010 16:07:34 +0000 (12:07 -0400)]
Fix MinGW toolchain detection
Updated the comment in change I6bef2ab5, but missed adding the code to
the commit.
Change-Id: I14d300489b79730e3995175bfe5f9271b569abe3
Yunqing Wang [Thu, 10 Jun 2010 15:48:48 +0000 (11:48 -0400)]
Improve vp8_sixtap_predict functions
Restructure vp8_sixtap_predict functions to eliminate extra 5-line
calculation while doing first-pass only. Also, combline functions
to eliminate usage of intermediate buffer. This gives decoder a 3%
performance gain on my test clips.
Change-Id: I13de49638884d1a57d0855c63aea719316d08c1b
John Koleszar [Thu, 10 Jun 2010 14:21:03 +0000 (07:21 -0700)]
Merge "Detect toolchain based on gcc -dumpmachine"
Philip Jägenstedt [Mon, 7 Jun 2010 04:12:14 +0000 (06:12 +0200)]
Detect toolchain based on gcc -dumpmachine
Using uname fails e.g. on a 64-bit machine with a 32-bit toolchain.
The following gcc -dumpmachine strings have been verified:
* 32-bit Linux gives i486-linux-gnu
* 64-bit Linux gives x86_64-linux-gnu
* Mac OS X 10.5 gives i686-apple-darwin9
* MinGW gives mingw32
*darwin8* and *bsd* can safely be assumed to be correct, but *cygwin*
is a guess.
Change-Id: I6bef2ab5e97cbd3410aa66b0c4f84d2231884b05
Paul Wilkins [Thu, 10 Jun 2010 10:24:54 +0000 (03:24 -0700)]
Merge "Adjust to avoid long line"
Paul Wilkins [Wed, 9 Jun 2010 14:03:48 +0000 (15:03 +0100)]
Adjust to avoid long line
Paul Wilkins [Thu, 10 Jun 2010 10:05:32 +0000 (03:05 -0700)]
Merge "Correct comment"
Paul Wilkins [Thu, 10 Jun 2010 10:04:45 +0000 (03:04 -0700)]
Merge "Fix RD bug."
John Koleszar [Thu, 10 Jun 2010 00:55:57 +0000 (17:55 -0700)]
Merge "Remove secondary mv clamping from decode stage"
John Koleszar [Thu, 10 Jun 2010 00:52:50 +0000 (17:52 -0700)]
Merge "Make shared object use extralibs"
John Koleszar [Wed, 9 Jun 2010 15:29:20 +0000 (11:29 -0400)]
Remove secondary mv clamping from decode stage
This patch removes the secondary MV clamping from the MV decoder. This
behavior was consistent with limits placed on non-split MVs by the
reference encoder, but was inconsistent with the MVs generated in the
split case.
The purpose of this secondary clamping was only to prevent crashes on
invalid data. It was not intended to be a behaviour an encoder could or
should rely on. Instead of doing additional clamping in a way that
changes the entropy context, the secondary clamp is removed and the
border handling is made implmentation specific. With respect to the
spec, the border is treated as essentially infinite, limited only by
the clamping performed on the near/nearest reference and the maximum
encodable magnitude of the residual MV.
This does not affect any currently produced streams.
Change-Id: I68d35a2fbb51570d6569eab4ad233961405230a3