Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(76)

Issue 1533643005: Add rounding to InterpolateRow for improved quality and consistency. (Closed)

Created:
5 years ago by fbarchard
Modified:
5 years ago
Reviewers:
harryjin, Diony Rosa
Base URL:
https://chromium.googlesource.com/libyuv/libyuv@master
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

Add rounding to InterpolateRow for improved quality and consistency. Remove inaccurate specializations for 1/4 and 3/4, since they round incorrectly. Specialize for 100% and 50% are kept due to performance. Make C and ARM code match SSSE3. Make unittests expect zero difference. BUG=libyuv:535 R=harryjin@google.com Committed: https://chromium.googlesource.com/libyuv/libyuv/+/f4447745ae646749562cff00e9d8fbc3b9043580

Patch Set 1 #

Patch Set 2 : fix 80 col lint warning #

Patch Set 3 : lint fix in unittest #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -512 lines) Patch
M include/libyuv/row.h View 3 chunks +0 lines, -7 lines 0 comments Download
M source/planar_functions.cc View 3 chunks +0 lines, -24 lines 0 comments Download
M source/row_any.cc View 1 chunk +0 lines, -3 lines 0 comments Download
M source/row_common.cc View 1 1 chunk +10 lines, -7 lines 0 comments Download
M source/row_gcc.cc View 12 chunks +14 lines, -175 lines 0 comments Download
M source/row_neon.cc View 4 chunks +5 lines, -36 lines 0 comments Download
M source/row_neon64.cc View 3 chunks +4 lines, -36 lines 0 comments Download
M source/row_win.cc View 11 chunks +12 lines, -164 lines 0 comments Download
M source/scale.cc View 2 chunks +0 lines, -16 lines 0 comments Download
M source/scale_argb.cc View 3 chunks +0 lines, -24 lines 0 comments Download
M source/scale_common.cc View 1 chunk +0 lines, -8 lines 0 comments Download
M unit_test/planar_test.cc View 1 2 3 chunks +12 lines, -12 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
fbarchard
arm code matches ssse3/avx2
5 years ago (2015-12-17 23:09:31 UTC) #2
harryjin
lgtm
5 years ago (2015-12-17 23:13:40 UTC) #3
fbarchard
93.00% libyuv_unittest libyuv_unittest [.] InterpolateRow_AVX2 ◆ 0.90% libyuv_unittest [kernel.kallsyms] [k] 0xffffffff810a7d10 ▒ 0.39% libyuv_unittest libc-2.19.so ...
5 years ago (2015-12-17 23:16:31 UTC) #4
fbarchard
5 years ago (2015-12-17 23:24:12 UTC) #6
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
f4447745ae646749562cff00e9d8fbc3b9043580 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698