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

Issue 344903003: libvpx: Remove edsp(armv5te) (Closed)

Created:
6 years, 6 months ago by fgalligan
Modified:
6 years, 6 months ago
Reviewers:
fgalligan1, Tom Finegan
CC:
chromium-reviews, wwcv, jzern, fgalligan1
Visibility:
Public.

Description

libvpx: Remove edsp(armv5te) The difference between the EDSP code and the C code is not measurable. This will also remove a dependency on asm_offsets. BUG=386801 TBR=tomfinegan@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278497

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -29 lines) Patch
M generate_gypi.sh View 1 chunk +4 lines, -4 lines 0 comments Download
M libvpx_srcs_arm.gypi View 1 chunk +1 line, -5 lines 0 comments Download
M libvpx_srcs_arm_neon.gypi View 2 chunks +1 line, -5 lines 0 comments Download
M libvpx_srcs_arm_neon_cpu_detect.gypi View 2 chunks +1 line, -5 lines 0 comments Download
M source/config/linux/arm-neon-cpu-detect/vpx_config.h View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm-neon-cpu-detect/vpx_config.asm View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm-neon-cpu-detect/vpx_config.c View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm-neon/vpx_config.h View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm-neon/vpx_config.asm View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm-neon/vpx_config.c View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm/vpx_config.h View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm/vpx_config.asm View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm/vpx_config.c View 1 chunk +1 line, -1 line 0 comments Download
M source/config/linux/arm64/vpx_config.c View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
fgalligan1
6 years, 6 months ago (2014-06-19 21:40:49 UTC) #1
fgalligan
Committed patchset #1 manually as r278497 (presubmit successful).
6 years, 6 months ago (2014-06-19 21:41:28 UTC) #2
Tom Finegan
6 years, 6 months ago (2014-06-19 21:42:25 UTC) #3
Message was sent while issue was closed.
On 2014/06/19 21:41:28, fgalligan wrote:
> Committed patchset #1 manually as r278497 (presubmit successful).

lgtm

Powered by Google App Engine
This is Rietveld 408576698