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

Issue 2084533006: YUV scale filter columns improved filtering accuracy (Closed)

Created:
4 years, 6 months ago by fbarchard1
Modified:
4 years, 6 months ago
Reviewers:
harryjin, wangcheng
CC:
wangcheng
Base URL:
https://chromium.googlesource.com/libyuv/libyuv@master
Target Ref:
refs/heads/master
Project:
libyuv
Visibility:
Public.

Description

YUV scale filter columns improved filtering accuracy upscale a YUV image. observe change in hue.. green especially. disable ScaleFilterCols_SSSE3, falling back on ScaleFilterCols_C observe hue.. green especially, is better. was ScaleFrom1280x720_Bilinear (1620 ms) now ScaleFrom1280x720_Bilinear (1907 ms) BUG=libyuv:605 TEST=try bots R=harryjin@google.com, wangcheng@google.com Committed: https://chromium.googlesource.com/libyuv/libyuv/+/cc88adc6209f3aa9c25adc6be02743fa2e9e9c80

Patch Set 1 #

Patch Set 2 : gcc version #

Patch Set 3 : handle odd width #

Patch Set 4 : port to windows #

Patch Set 5 : tighten up all bilinear tests #

Patch Set 6 : git cl lint #

Patch Set 7 : more lint fixes #

Patch Set 8 : cast xmmword ptr for clangcl #

Patch Set 9 : git cl try #

Patch Set 10 : clang fix pixel 2 loop #

Patch Set 11 : bump version #

Unified diffs Side-by-side diffs Delta from patch set Stats (+88 lines, -32 lines) Patch
M README.chromium View 1 2 3 4 5 6 7 8 9 10 1 chunk +1 line, -1 line 0 comments Download
M include/libyuv/scale_row.h View 1 chunk +1 line, -2 lines 0 comments Download
M include/libyuv/version.h View 1 2 3 4 5 6 7 8 9 10 1 chunk +1 line, -1 line 0 comments Download
M source/scale_common.cc View 1 2 3 4 5 6 3 chunks +12 lines, -2 lines 0 comments Download
M source/scale_gcc.cc View 1 2 3 5 chunks +38 lines, -12 lines 0 comments Download
M source/scale_neon.cc View 1 2 3 1 chunk +4 lines, -0 lines 0 comments Download
M source/scale_win.cc View 1 2 3 4 5 6 7 8 4 chunks +26 lines, -9 lines 0 comments Download
M unit_test/scale_test.cc View 1 2 3 4 2 chunks +5 lines, -5 lines 0 comments Download

Messages

Total messages: 12 (6 generated)
fbarchard1
improved quality YUV scaling with more accurate column code.
4 years, 6 months ago (2016-06-22 17:34:47 UTC) #3
harryjin
lgtm
4 years, 6 months ago (2016-06-22 17:49:57 UTC) #4
fbarchard1
clangcl on windows has a build error? ninja -t msvc -e environment.x86 -- E:\b\build\goma/gomacc "..\..\third_party/llvm-build/Release+Asserts/bin/clang-cl" ...
4 years, 6 months ago (2016-06-22 19:28:28 UTC) #6
fbarchard1
reported clang-cl issue https://llvm.org/bugs/show_bug.cgi?id=28266
4 years, 6 months ago (2016-06-22 23:29:31 UTC) #7
wangcheng
lgtm
4 years, 6 months ago (2016-06-23 22:56:59 UTC) #10
fbarchard1
4 years, 6 months ago (2016-06-24 03:17:00 UTC) #12
Message was sent while issue was closed.
Committed patchset #11 (id:200001) manually as
cc88adc6209f3aa9c25adc6be02743fa2e9e9c80 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698